Comments for constants #1973

pull sipa wants to merge 1 commits into bitcoin:master from sipa:comconst changing 1 files +14 −1

sipa commented at 11:15 PM on November 2, 2012: member

No functional changes.
Comments for constants 9d6633ac0d
gavinandresen commented at 11:19 PM on November 2, 2012: contributor

ACK
Diapolo commented at 11:30 PM on November 2, 2012: none

That's a good thing!
gmaxwell commented at 3:01 AM on November 3, 2012: contributor

Violations of MAX_INV_SZ get you DOSflagged. So it should get some "P2P node rule" flag, perhaps? MAX_MONEY is an implicit protocol rule. Kinda obvious though. ACK in any case.
BitcoinPullTester commented at 5:43 AM on November 3, 2012: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/9d6633ac0de9d2a24333525cffdf0b23164ecad3 for binaries and test log.
sipa referenced this in commit 7fac57786b on Nov 3, 2012
sipa merged this on Nov 3, 2012
sipa closed this on Nov 3, 2012
laudney referenced this in commit 43b3548718 on Mar 19, 2014
DrahtBot locked this on Sep 8, 2021

Contributors