Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] Remove useless if-before-free tests.

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:14

Jim Meyering [off-list ref] writes:
It's a definite loss of portability if you can find a reasonable porting
target for which free(NULL) fails.  But even if you do, the fix is
not to reject the clean-up, but to amend it with a wrapper function.
That encapsulates the work-around in one place rather than polluting
all of those files.
As we already have unchecked free(ptr) in our code _anyway_,
there _technically_ is no reason to reject the clean-up patch.

We just need to find a quiescent time to do so so that actively
cooking patches in people's trees (and topics in 'next') won't
get needless conflicts.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help