Page MenuHomePhabricator

Removed activation logic for OP_REVERSEBYTES to pretend it was always enabled
ClosedPublic

Authored by tobias_ruck on Wed, May 20, 18:56.

Details

Summary

Activation logic is now unnecessary, assuming this code doesn't travel back in time.
This change removes the scriptflag SCRIPT_ENABLE_OP_REVERSEBYTES and all the places it gets activated.
Also entirely removes the activation tests.

Test Plan

ninja check
ninja check-functional

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

tobias_ruck created this revision.Wed, May 20, 18:56
Owners added a reviewer: Restricted Owners Package.Wed, May 20, 18:56
Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, May 20, 18:56
tobias_ruck requested review of this revision.Wed, May 20, 18:56
jasonbcox accepted this revision.Wed, May 20, 21:26
jasonbcox added a subscriber: jasonbcox.

Thanks for cleaning this up :)

This revision is now accepted and ready to land.Wed, May 20, 21:26