Use -Wall with relevant -Wno and fix one operator undefined error. #282

pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:wall changing 2 files +14 −2
  1. TheBlueMatt commented at 7:58 PM on May 28, 2011: member

    The goal here is to show people how many warnings we have and get someone to look into them. Additionally, this might prevent keep more warnings from popping up with new patches (though thats a bit unlikely).

  2. Use -Wall with relevant -Wno and fix one operator undefined error.
    The goal here is to show people how many warnings we have and
    get someone to look into them.  Additionally, this might prevent
    keep more warnings from popping up with new patches (though
    thats a bit unlikely).
    5d9f459370
  3. jgarzik commented at 8:59 AM on June 14, 2011: contributor

    non-committal (neither NAK nor ACK)...

  4. TheBlueMatt commented at 10:07 AM on June 14, 2011: member

    Meaning?

  5. jgarzik commented at 5:04 PM on June 14, 2011: contributor

    heh. Well it's a long list that I'm not sure will be of much value over current setup. Fixing warnings can be useful, or it can just be a compiler being dumb and we're working around it. So if Gavin or sipa or tcatm wants this pull, I would not object, but I'm not gonna click 'merge' myself...

  6. TheBlueMatt commented at 10:22 PM on June 14, 2011: member

    Ah, well I suppose it was half a pull to hopefully bring attention to some (potential) bugs we might have. Also, it would have fixed a coding mistake or two of mine...but that is really just because Im in idiot.

  7. jgarzik commented at 5:49 AM on June 20, 2011: contributor

    closing the half-a-pull :)

  8. jgarzik closed this on Jun 20, 2011

  9. sipa referenced this in commit 0350c7e4b8 on Jul 28, 2015
  10. sipa referenced this in commit 7863d105ea on Aug 4, 2015
  11. sipa referenced this in commit 780d5b3c7b on Aug 23, 2015
  12. TheBlueMatt referenced this in commit a671356e1f on Oct 20, 2015
  13. sipa referenced this in commit 9177950c74 on Oct 21, 2015
  14. sipa referenced this in commit f4787d1caf on Oct 21, 2015
  15. sipa referenced this in commit 6557a8cd46 on Oct 26, 2015
  16. sipa referenced this in commit ea06490d14 on Oct 27, 2015
  17. sipa referenced this in commit 003bb87153 on Nov 5, 2015
  18. sipa referenced this in commit bfd83199c3 on Nov 11, 2015
  19. sipa referenced this in commit b437ea7ec9 on Nov 12, 2015
  20. sipa referenced this in commit 1d84107924 on Nov 12, 2015
  21. dexX7 referenced this in commit 0d7260eb75 on Nov 13, 2015
  22. rebroad referenced this in commit 40ead34fbe on Dec 7, 2016
  23. deadalnix referenced this in commit c33307495b on Jan 19, 2017
  24. classesjack referenced this in commit 86e47d1d63 on Jan 2, 2018
  25. attilaaf referenced this in commit f1cfadd3da on Jan 13, 2020
  26. DrahtBot locked this on Sep 8, 2021
Contributors

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-24 15:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me