On Tue, 16 Jun 2015 16:29:08 -0400, Andrei Alexandrescu
<seewebsiteforem...@erdani.org> wrote:
On Saturday, 13 June 2015 at 19:08:26 UTC, Jacob Carlborg wrote:
On 2015-06-12 13:06, Dicebot wrote:
The legendary allocator package by Andrei Alexandrescu has arrived at
your doorsteps and kindly asks to let it into Phobos
http://wiki.dlang.org/Review/std.experimental.allocator
Docs:
http://erdani.com/d/phobos-prerelease/std_experimental_allocator.html
Code:
https://github.com/andralex/phobos/tree/allocator/std/experimental/allocator
I think "IAllocator", "theAllocator" and "it" are really bad names. I
recommend renaming those symbols to:
IAllocator -> Allocator
theAllocator -> currentAllocator or tlsAllocator
it -> allocator or instance. Even better if a static "opDispatch" could
be used to forward all methods to the instance.
https://github.com/D-Programming-Language/phobos/commit/319f3297418c515a6d2e52e6e52d0f3f5895f587
changes .it to .instance for all allocators. -- Andrei
Why not .that? ;)
Bit