On Wednesday, 26 February 2014 at 15:00:13 UTC, Dicebot wrote:
On Wednesday, 26 February 2014 at 04:43:30 UTC, Mike wrote:
Please review this (http://dpaste.dzfl.pl/2377217c7870)
instead.
throw new Exception("Out of memory");
I don't think it is a good thing to do. Of course GC is likely
to have some memory left in hs internal pools but in general
once you hit out-of-memory state last thing you want is not
allocate even more.
// Deallocate test
heapDeallocate(test);
More idiomatic D is to put `scope(exit) heapDeallocate(test);`
immediately after allocation line.
Thanks,
I've made changes and I guess this is the one I'll use for the
pull.
(http://dpaste.dzfl.pl/31367860c005)