Alex Riesen <raa.lkml@xxxxxxxxx> wrote: > Jim Meyering, Tue, Aug 19, 2008 20:46:30 +0200: >> We removed a handful of these useless if-before-free tests >> several months ago. This change removes a new one that snuck back in. > > Something used to crash in free when passed NULL. It's one of the > reasons why some cross-platform projects have xfree which does the > check (sometimes depending on platform). Admittedly, it is a long time > since I saw such a platform (some old SunOS it was, I believe). Right. This was discussed at length back in February: http://thread.gmane.org/gmane.comp.version-control.git/74187 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html