ensure we use our format macros to avoid compilation warnings #2018

Merged
merged 1 commit into from Nov 16, 2012

Conversation

Projects
None yet
3 participants

Diapolo commented Nov 16, 2012

  • fixes 2 warnings I observed while compiling on Windows with MinGW
ensure we use our format macros to avoid compilation warnings
- fixes 2 warnings I observed while compiling on Windows with MinGW

sipa added a commit that referenced this pull request Nov 16, 2012

Merge pull request #2018 from Diapolo/fix_warning
ensure we use our format macros to avoid compilation warnings

@sipa sipa merged commit 1fefd2a into bitcoin:master Nov 16, 2012

%lld is always correct for long long type, per C specifications... so you shouldn't have gotten a warning :/

PRI64u is for unsigned, but GetValueOut/GetBlockValue return signed...

Diapolo pushed a commit to Diapolo/bitcoin that referenced this pull request Nov 17, 2012

ConnectBlock(): fix error() format to be unsigned
- I introduced the wrong format macro with my former patch (#2018), this
  needs to be signed not unsigned (thanks Luke-Jr)

laanwj added a commit that referenced this pull request Nov 17, 2012

gavinandresen pushed a commit that referenced this pull request Dec 10, 2012

ConnectBlock(): fix error() format to be unsigned
- I introduced the wrong format macro with my former patch (#2018), this
  needs to be signed not unsigned (thanks Luke-Jr)

laudney pushed a commit to reddcoin-project/reddcoin that referenced this pull request Mar 19, 2014

Merge pull request #2018 from Diapolo/fix_warning
ensure we use our format macros to avoid compilation warnings

laudney pushed a commit to reddcoin-project/reddcoin that referenced this pull request Mar 19, 2014

ConnectBlock(): fix error() format to be unsigned
- I introduced the wrong format macro with my former patch (#2018), this
  needs to be signed not unsigned (thanks Luke-Jr)

laudney pushed a commit to reddcoin-project/reddcoin that referenced this pull request Mar 19, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment