On 10/24/2013 09:54 PM, Andrei Alexandrescu wrote:
Hello,


I know it's been a long wait. Hopefully it was worth it. The alpha release of 
untyped allocators is ready for tire-kicking and a test drive.

Code: https://github.com/andralex/phobos/blob/allocator/std/allocator.d

Dox: http://erdani.com/d/phobos-prerelease/std_allocator.html

Looks good.

...

Please destroy! I've literally sweat as I'm sending this :o).
...

- Possible null dereference on line 3128. :o)
(In general, CascadingAllocator is not yet very composable. It should return null allocations instead of failing in undocumented ways. It should probably also allow the 'make' function to return null.)

- "CAllocator". The name is non-descriptive/misleading.

- Preconditions should be in the corresponding section.

Reply via email to