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

Re: [PATCH 4/4] Fix unreachable-code warning with clang on Windows

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 4, 2025, 15:50 UTC
Message-ID
<xmqq8qm7cxq3.fsf@gitster.g>
In-Reply-To
<aD_3ahX2jyrtfvjq@pks.im>
Patrick Steinhardt <ps@pks.im> writes:
> The function won't cause the error, but the macro will. So wouldn't it
> make more sense to wrap the macro itself in `NOT_CONSTANT`, like this:
>
>     #define create_ref_symlink(a, b) NOT_CONSTANT(-1)
That's clever ;-).

We cannot unfortunatel do the same at the site that the macro NOT_CONSTANT() was invented for, though.

Previous: Patrick SteinhardtNext: Mike Hommey
Message 4 of 11 in “Fix maybe-uninitialized warning with GCC at -O3”
  1. 1/4 Fix maybe-uninitialized warning with GCC at -O3Mike Hommey, Jun 3, 2025
  2. 4/4 Fix unreachable-code warning with clang on WindowsMike Hommey, Jun 3, 2025
  3. Patrick SteinhardtJun 4, 2025
  4. Junio C HamanoJun 4, 2025
  5. 3/4 Fix comma warnings with clang on WindowsMike Hommey, Jun 3, 2025
  6. Junio C HamanoJun 3, 2025
  7. 2/4 Fix use-after-free warning with GCC at -O3Mike Hommey, Jun 3, 2025
  8. Junio C HamanoJun 3, 2025
  9. Patrick SteinhardtJun 4, 2025
  10. Patrick SteinhardtJun 4, 2025
  11. Jeff KingJun 6, 2025

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.