diff options
| author | Ava Chow <github@achow101.com> | 2024-05-03 12:36:56 -0400 |
|---|---|---|
| committer | Ava Chow <github@achow101.com> | 2024-05-03 12:36:56 -0400 |
| commit | f5b6f621fff7540ca97974b26a66ca1cc6db018c (patch) | |
| tree | 911c114e6f8b7bb9ef2ba71831f5f22b8bba0d06 /test | |
| parent | f9486de6a96b624982a2d45b895bfe4e865063d1 (diff) | |
| parent | ffc674595cb19b2fdc5705b355bdd3e7f724b860 (diff) | |
| download | bitcoin-f5b6f621fff7540ca97974b26a66ca1cc6db018c.tar.xz | |
Merge bitcoin/bitcoin#30024: doc: replace remaining "520" magic nums with MAX_SCRIPT_ELEMENT_SIZE
ffc674595cb19b2fdc5705b355bdd3e7f724b860 Replace remaining "520" magic numbers with MAX_SCRIPT_ELEMENT_SIZE (Jon Atack)
Pull request description:
Noticed these while reviewing BIPs yesterday.
It would be clearer and more future-proof to refer to their constant name.
ACKs for top commit:
instagibbs:
ACK ffc674595cb19b2fdc5705b355bdd3e7f724b860
sipa:
ACK ffc674595cb19b2fdc5705b355bdd3e7f724b860
achow101:
ACK ffc674595cb19b2fdc5705b355bdd3e7f724b860
glozow:
ACK ffc674595cb19b2fdc5705b355bdd3e7f724b860, agree it's clearer for these comments to refer to the greppable name of the limit rather than the number
Tree-SHA512: 462afc1c64543877ac58cb3acdb01d42c6d08abfb362802f29f3482d75401a2a8adadbc2facd222a9a9fefcaab6854865ea400f50ad60bec17831d29f7798afe
Diffstat (limited to 'test')
| -rwxr-xr-x | test/functional/p2p_segwit.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/functional/p2p_segwit.py b/test/functional/p2p_segwit.py index 213c748eda..45bbd7f1c3 100755 --- a/test/functional/p2p_segwit.py +++ b/test/functional/p2p_segwit.py @@ -1054,7 +1054,7 @@ class SegWitTest(BitcoinTestFramework): @subtest def test_max_witness_push_length(self): - """Test that witness stack can only allow up to 520 byte pushes.""" + """Test that witness stack can only allow up to MAX_SCRIPT_ELEMENT_SIZE byte pushes.""" block = self.build_next_block() |