this post was submitted on 20 Jun 2026
41 points (90.2% liked)

Technology

85567 readers
3772 users here now

This is a most excellent place for technology news and articles.


Our Rules


  1. Follow the lemmy.world rules.
  2. Only tech related news or articles.
  3. Be excellent to each other!
  4. Mod approved content bots can post up to 10 articles per day.
  5. Threads asking for personal tech support may be deleted.
  6. Politics threads may be removed.
  7. No memes allowed as posts, OK to post as comments.
  8. Only approved bots from the list below, this includes using AI responses and summaries. To ask if your bot can be added please contact a mod.
  9. Check for duplicates before posting, duplicates may be removed
  10. Accounts 7 days and younger will have their posts automatically removed.

Approved Bots


founded 3 years ago
MODERATORS
top 6 comments
sorted by: hot top controversial new old
[–] ProfessorScience@lemmy.world 6 points 3 hours ago (2 children)

Another option is to disable the warning locally [...] But this is compiler-specific and noisy. It also does not explain the real issue to anyone reading the code.

I don't understand this logic. Surely disabling the warning is directly explaining the actual issue, that the code emits a false positive warning for specific compilers and conditions? That the false_but_compiler_does_not_know_it_ thing is apparently the better way to convey what's going on seems kind of ridiculous.

You could say that the current way is so unclear that the guy had to write a 2 thousand word essay to explain it.

[–] Venator@lemmy.nz 2 points 49 minutes ago* (last edited 45 minutes ago) (1 children)

My guess would be they didn't like how verbose it was to supress the warning, or maybe they have a coding standard not to suppress warnings, and this was a workaround for that 😅

[–] Venator@lemmy.nz 1 points 31 minutes ago* (last edited 27 minutes ago)

actually not sure, as they do use #pragma GCC diagnostic ignored in a couple of places :

https://github.com/git/git/blob/8d96f09e9245ddf80c1981476fcbac8c4bb4125f/compat/win32/headless.c#L14

https://github.com/git/git/blob/8d96f09e9245ddf80c1981476fcbac8c4bb4125f/compat/regex/regex.c#L20

but apparently that disables it for everything after that line in the file though:

https://stackoverflow.com/questions/48426484/concise-way-to-disable-specific-warning-instances-in-clang#comment83878587_48426846

I think the second answer on that stackoverflow post might be better than false_but_the_compiler_does_not_know_it_ though...

[–] Dialectical_Specialist@quokk.au 4 points 4 hours ago

I disagree on the naming of 'compiler-tricks'. Some of us are complete idiots and prefer the compiler trick US

​^(^​^N^​^o^​^t^​^t^​^h^​^e^​^o^​^t^​^h^​^e^​​^r^​^w^​^a^​^y^​^a^​^r^​^o^​^u^​^n^​^d^​^.^​^)^

[–] sun_is_ra@sh.itjust.works 5 points 5 hours ago

Good read!

A small UI issue: the screenshot appear in darkmode but I could find how to trigger the dark mode in the website.