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

Re: [PATCH] Makefile: suppress false positive warnings of empty format string.

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Sep 30, 2013, 21:26 UTC
Message-ID
<20130930212636.GY9464@google.com>
In-Reply-To
<20130930201429.GA14433@sigill.intra.peff.net>
Jeff King wrote:
>                                            I think we'd be better off to
> simply mark up the few callsites. Workarounds are here:
>
>   http://article.gmane.org/gmane.comp.version-control.git/230026

Yeah, that looks okay (ugly but tolerable). It's tempting to do something like

	-	status_printf_ln(s, GIT_COLOR_NORMAL, "");
	+	status_nl(s, GIT_COLOR_NORMAL);
and
	-	status_printf(s, color(WT_STATUS_HEADER, s), "");
	+	status_start_line(s, color(WT_STATUS_HEADER, s));
to make the intent clearer.  Sane?
Jonathan
Previous: Jeff KingNext: Jeff King
Message 7 of 10 in “What's cooking in git.git (Jul 2013, #04; Thu, 11)”
  1. Junio C HamanoJul 11, 2013
  2. Makefile: suppress false positive warnings of empty format string.Stefan Beller, Sep 29, 2013
  3. Felipe ContrerasSep 29, 2013
  4. Ramsay JonesSep 29, 2013
  5. Jonathan NiederSep 29, 2013
  6. Jeff KingSep 30, 2013
  7. Jonathan NiederSep 30, 2013
  8. Jeff KingSep 30, 2013
  9. Stefan BellerSep 30, 2013
  10. Jeff KingSep 30, 2013

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.