Remove confusing MAX_BLOCK_BASE_SIZE.
commit3babbcb48786372d4b22171674c4cc5a6220c294
authorGregory Maxwell <greg@xiph.org>
Sat, 17 Jun 2017 00:18:42 +0000 (17 00:18 +0000)
committerGregory Maxwell <greg@xiph.org>
Fri, 14 Jul 2017 19:24:17 +0000 (14 19:24 +0000)
tree099fab1403b69a6da62d222fbf8b2d45f0c12fa7
parente4fcbf797ed3b472d352ac3794ec82f581209c50
Remove confusing MAX_BLOCK_BASE_SIZE.

Some people keep thinking that MAX_BLOCK_BASE_SIZE is a separate
 size limit from the weight limit when it fact it is superfluous,
 and used in early tests before the witness data has been
 validated or just to compute worst case sizes.  The size checks
 that use it would not behave any differently consensus wise
 if they were eliminated completely.

Its correct value is not independently settable but is a function
 of the weight limit and weight formula.

This patch just eliminates it and uses the scale factor as
 required to compute the worse case constants.

It also moves the weight factor out of primitives into consensus,
 which is a more logical place for it.
17 files changed:
src/bitcoin-tx.cpp
src/blockencodings.cpp
src/coins.cpp
src/consensus/consensus.h
src/consensus/tx_verify.cpp
src/consensus/validation.h
src/core_write.cpp
src/merkleblock.cpp
src/policy/policy.cpp
src/primitives/block.cpp
src/primitives/block.h
src/primitives/transaction.cpp
src/primitives/transaction.h
src/rpc/mining.cpp
src/test/sigopcount_tests.cpp
src/undo.h
src/validation.cpp