Cleanups suggested by genjix #716

pull gavinandresen wants to merge 3 commits into bitcoin:master from gavinandresen:cleanup changing 15 files +35 −78

gavinandresen commented at 12:20 AM on December 20, 2011: contributor
Several miscellaneous code cleanups:
- Removed all mention of BORLAND_C : we don't (and never have, as far as I know) support that compiler.
- All modern C++ compilers support 'long long' for 64-bit integers, so removed the windows #ifdefs
- Replaced all occurrences of INT_MAX/etc with the typesafe std::numeric_limits<> methods
- Replaced a constant 10,000-byte buffer with a properly ECDSA_size'ed vector (and removed a memcpy)
Cleanup: removed dead code, and use C99 typedefs for int64 (supported by all modern c++ compilers) bd846c0e56
Use std::numeric_limits<> for typesafe INT_MAX/etc 26ce92b352
Code cleanup: use ECDSA_size() instead of fixed 10,000 byte sig buffer, and explicity init static var 9ef7fa3447
TheBlueMatt commented at 1:19 AM on December 20, 2011: member

ACK
gavinandresen referenced this in commit 74f5435e10 on Dec 20, 2011
gavinandresen merged this on Dec 20, 2011
gavinandresen closed this on Dec 20, 2011
coblee referenced this in commit fea5ea8305 on Jul 17, 2012
destenson referenced this in commit 4474de8573 on Jun 26, 2016
destenson referenced this in commit 73ff4b7235 on Aug 8, 2017
dexX7 referenced this in commit deeb085877 on Aug 13, 2018
Losangelosgenetics referenced this in commit 0e1a03575e on Mar 12, 2020
DrahtBot locked this on Sep 8, 2021

Contributors