www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 18849] New: std.allocator: AllocatorList uses deallocate and


          Issue ID: 18849
           Summary: std.allocator: AllocatorList uses deallocate and
                    ignores return value in deallocateAll
           Product: D
           Version: D2
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P1
         Component: phobos
          Assignee: nobody puremagic.com
          Reporter: dlang-bugzilla thecybershadow.net

AllocatorList's deallocateAll has the code:

        if (special)

This has the issues:

- The return value is ignored. If the deallocation failed, the failure should
be communicated to the caller. Currently, this causes a silent memory leak.

- Some allocators, such as Region, can deallocateAll, but cannot deallocate.
Ideally the complete inability to deallocate() should be detected at
compile-time, and such combinations rejected.

- The documentation of AllocatorList has two examples of using it with Region,
which will not work (leak memory) due to the above.

May 10 2018