Arena::insert(impl Into<T>)

LPGhatguy / thunderdome

Arena type inspired by generational-arena

Apache License 2.0

197 stars 15 forks source link

Arena::insert(impl Into<T>) #20

Closed toyboot4e closed 3 years ago

toyboot4e commented 3 years ago

impl Into<T> is good for usability, but not good for compilation time.

I'm opening issues these days, but they are actually questions. If thuderdome become the best arena crate (TM), that's both nice and good for my learning :)

LPGhatguy commented 3 years ago

Thanks for the suggestion on this!

I think the bigger trade-off with this is that impl Into<T> can cause a lot of implicit coercions. I think that for a low-level container library like Thunderdome, it's important to keep its API explicit and free of implicit behavior.