I noticed that we've run out of bits in script flags, so I refactored the code so flags are an explicit class and cause compile failures instead of silent bugs, and can be safely extended to larger sizes. IMO it is also more readable, especially in regard to IsAtLeastOneSet.
Refactor script interpreter flags to use new CScriptFlags class for stricter checking #10353
pull luke-jr wants to merge 1 commits into bitcoin:master from luke-jr:scriptflags changing 13 files +230 −169-
luke-jr commented at 3:01 AM on May 8, 2017: member
-
Refactor script interpreter flags to use new CScriptFlags class for stricter checking 664299ec3b
- fanquake added the label Refactoring on May 8, 2017
- fanquake added the label Validation on May 8, 2017
-
jtimon commented at 1:04 AM on September 6, 2017: contributor
Needs rebase
-
dcousens commented at 3:54 AM on September 6, 2017: contributor
concept ACK
-
laanwj commented at 6:58 PM on March 7, 2018: member
Closing, seems to be dormant, let me know if you pick this up again.
- laanwj closed this on Mar 7, 2018
- MarcoFalke locked this on Sep 8, 2021
Labels