Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Comments for constants #1973

Merged
merged 1 commit into from Nov 3, 2012

Conversation

Projects
None yet
5 participants
Owner

sipa commented Nov 2, 2012

No functional changes.

Contributor

gavinandresen commented Nov 2, 2012

ACK

Diapolo commented Nov 2, 2012

That's a good thing!

Member

gmaxwell commented Nov 3, 2012

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.

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/9d6633ac0de9d2a24333525cffdf0b23164ecad3 for binaries and test log.

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

@sipa sipa Merge pull request #1973 from sipa/comconst
Comments for constants
7fac577

@sipa sipa merged commit 7fac577 into bitcoin:master Nov 3, 2012

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

@sipa sipa Merge pull request #1973 from sipa/comconst
Comments for constants
43b3548
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment