git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Make git-fmt-merge-msg a builtin

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jul 3, 2006, 21:29 UTC
Message-ID
<Pine.LNX.4.63.0607032327470.29667@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To
<20060703191635.21ba0af3.tihirvon@gmail.com>
Hi,
On Mon, 3 Jul 2006, Timo Hirvonen wrote:
> Seems that C89 requires free(NULL) to be a no-op but on some old systems 
> (SunOS) it may crash.  IMNSHO these systems were designed to crash valid 
> programs and torture developers.
At least it is not Malbolge. Or even VAX. (In that order.)
> There are probably many free(NULL) and realloc(NULL, ...) uses in the 
> git source code and are not worth fixing.

AFAIK realloc(NULL, ...) was fine even with K&R, whereas free(NULL) poses problems. Anyway, I do not _want_ to say that NULL should be free()d, because it just sounds wrong.

Ciao, Dscho

Previous: Timo Hirvonen
Message 7 of 7 in “Make git-fmt-merge-msg a builtin”
  1. Make git-fmt-merge-msg a builtinJohannes Schindelin, Jul 3, 2006
  2. Timo HirvonenJul 3, 2006
  3. Johannes SchindelinJul 3, 2006
  4. Timo HirvonenJul 3, 2006
  5. Johannes SchindelinJul 3, 2006
  6. Timo HirvonenJul 3, 2006
  7. Johannes SchindelinJul 3, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.