LLu is standard, but not portable. use ULL #4999

pull sinetek wants to merge 1 commits into bitcoin:master from sinetek:master changing 1 files +1 −1
  1. sinetek commented at 10:24 PM on September 28, 2014: contributor

    LLu is standard, but not portable with msvc. use ULL.

  2. LLu is standard, but not portable. use ULL 775b7b8d70
  3. sipa commented at 3:29 AM on September 29, 2014: member

    utACK

  4. luke-jr commented at 3:32 AM on September 29, 2014: member

    Not sure we should spend time supporting compilers that can't handle standard C++...

  5. sinetek commented at 6:23 AM on September 29, 2014: contributor

    Oh I get that, I am not expecting others do the leg work :-) I like Unix as much as the next dev; this however is a low-hanging fruit that's easy to pick, without having to introduce dialect madness.

  6. laanwj commented at 7:19 AM on September 29, 2014: member

    Well, ULL is more consistent with the rest of the code as well, so ACK.

  7. laanwj merged this on Sep 29, 2014
  8. laanwj closed this on Sep 29, 2014

  9. laanwj referenced this in commit 76182e7cb6 on Sep 29, 2014
  10. DrahtBot locked this on Sep 8, 2021

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-13 21:15 UTC

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