From: Matthieu Moy Date: Tue, 17 May 2016 09:05:06 GMT Subject: Re: [RFD PATCH 0/3] Free all the memory! Message-ID: In-Reply-To: <20160517032247.25092-1-sbeller@google.com> Stefan Beller writes: > The end goal of this (unfinished) series is to close all intentional memory > leaks when enabling the -DFREE_ALL_MEMORY switch. This is just > demonstrating how the beginning of such a series could look like. One potential issue with this is: if all developers and the testsuite use this -DFREE_ALL_MEMORY, the non-free-all-memory setup will not be well tested, and still this is the one used by real people. For example, if there's a really annoying memory leak hidden by FREE_ALL_MEMORY, we may not notice it. Perhaps it'd be better to activate FREE_ALL_MEMORY only when tools like valgrind or so is used. -- Matthieu Moy http://www-verimag.imag.fr/~moy/