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

Re: [PATCH] config.mak.dev: enable -Wunused-function

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 19, 2018, 01:23 UTC
Message-ID
<xmqqftx24tlc.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<3295c759-473b-f279-5443-f915fd0b1061@ramsayjones.plus.com>
Ramsay Jones <ramsay@ramsayjones.plus.com> writes:
> I haven't looked too deeply, but this seems to be caused by
> Junio's commit 42c89ea70a ("SQUASH??? - convert the other user of
> string-list as db", 2018-10-17) which removes a call to the
> add_existing() function - the subject of the warning.

That is very understandable and it is immediately obvious to me that the unused-function warning is being useful there ;-)

Previous: Ramsay Jones
Message 13 of 13 in “config.mak.dev: enable -Wunused-function”
  1. config.mak.dev: enable -Wunused-functionJeff King, Oct 18, 2018
  2. Jeff KingOct 18, 2018
  3. Duy NguyenOct 18, 2018
  4. Jeff KingOct 18, 2018
  5. thread-utils: better wrapper to avoid #ifdef NO_PTHREADSNguyễn Thái Ngọc Duy, Oct 18, 2018
  6. Jeff KingOct 23, 2018
  7. Junio C HamanoOct 24, 2018
  8. Ben PeartOct 26, 2018
  9. can we deprecate NO_PTHREADS?, was: better wrapper to avoid #ifdef NO_PTHREADSJeff King, Oct 27, 2018
  10. Duy NguyenOct 27, 2018
  11. Jeff KingOct 27, 2018
  12. Ramsay JonesOct 18, 2018
  13. Junio C HamanoOct 19, 2018

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.