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

Re: [PATCH 2/3] Fix some "variable might be used uninitialized" warnings

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 11, 2011, 20:48 UTC
Message-ID
<7vpqj6olfa.fsf@alter.siamese.dyndns.org>
In-Reply-To
<4E6D0E74.1020801@ramsay1.demon.co.uk>
Ramsay Jones <ramsay@ramsay1.demon.co.uk> writes:
Show 11 quoted lines
> In particular, gcc complains as follows:
>
>         CC tree-walk.o
>     tree-walk.c: In function `traverse_trees':
>     tree-walk.c:347: warning: 'e' might be used uninitialized in this \
>         function
>
>         CC builtin/revert.o
>     builtin/revert.c: In function `verify_opt_mutually_compatible':
>     builtin/revert.c:113: warning: 'opt2' might be used uninitialized in \
>         this function
Could you also add something to this effect to the commit log message:
	but I have verified that these are gcc being not careful
	enough and they are never used uninitialized.
If that is what you indeed have done, that is.
Thanks.
Previous: Ramsay JonesNext: Ramsay Jones
Message 2 of 5 in “Fix some "variable might be used uninitialized" warnings”
  1. 2/3 Fix some "variable might be used uninitialized" warningsRamsay Jones, Sep 11, 2011
  2. Junio C HamanoSep 11, 2011
  3. Ramsay JonesSep 13, 2011
  4. Ramsay JonesOct 8, 2011
  5. Junio C HamanoOct 10, 2011

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.