Thread (82 messages) flat view 82 messages, 11 authors, 2021-09-23

Re: UNLEAK(), leak checking in the default tests etc.

From: Felipe Contreras <hidden>
Date: 2021-06-09 20:36:40

Andrzej Hunt wrote:
On 09/06/2021 16:38, Ævar Arnfjörð Bjarmason wrote:
quoted
I wonder if that shouldn't be in DEVOPTS (or even a default under
DEVELOPER=1). I.e. you don't need any other special compile flags, just
a compiled git that you then run under valgrind to spot this.
I'm not familiar with git's development conventions/philosophy, but my 
2c is that it's better not to enable it by default in order to minimise 
divergence from the code that users are running.
It woudln't be on by default, you would need to turn it on with
`make DEVEOPER=1`.

-- 
Felipe Contreras
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help