Implement `Allocator` for `Bump`, not `&Bump`
Bumpdoes not implementAllocatorbut&Bumpdoes. So to get a&dyn Allocatoryou'd have to take a reference to the reference:&&bump_arena as &dyn AllocatorThat being said, I think the
impl Allocator for &Bumpcould be changed toimpl Allocator for Bumpso you wouldn't have that double indirection.
Originally posted by @bluurryy in #280
I don't think this would be a breaking change. How is it breaking?
EDIT: since there is a blanket impl Allocator for all &T where T: Allocator
EDIT: since there is a blanket impl
Allocatorfor all&TwhereT: Allocator
Yep. Although maybe removing the explicit import for &Bump and allowing the blanket impl to fill that in means this wouldn't be a breaking change. Not totally sure.