Phobos addition formal review: std.experimental.allocator

[bitwise via Digitalmars-d]

On Tue, 16 Jun 2015 16:29:08 -0400, Andrei Alexandrescu  
<SeeWebsiteForEmail at 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