Page MenuHomePhabricator

OP_INVERT implementation
AbandonedPublic

Authored by danconnolly on Jul 26 2018, 23:46.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

Implementation of the OP_INVERT opcode for the magnetic upgrade. The specification for this opcode is here.

Test Plan

make check

Diff Detail

Branch
opcodes_invert
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2987
Build 4070: arc lint + arc unit

Event Timeline

the reorganiztion of script_tests.json shouldnt be in this diff

danconnolly edited the summary of this revision. (Show Details)

refactor the diffs to be more coherent and scope refined

There should be some additional testing in the magnetic anomaly activation test.

See here that there's already a test landed: https://reviews.bitcoinabc.org/rABC4714cd3622565b35d08fa71d932482ad760cc0ba

There should be some additional testing in the magnetic anomaly activation test.

See here that there's already a test landed: https://reviews.bitcoinabc.org/rABC4714cd3622565b35d08fa71d932482ad760cc0ba

Yes, I see. Those tests didn't exist when we started this but I see they would be good to have. We would need to add tests for each opcode being implemented. The code of the tests doesn't seem very extendable, I think we'd need to refactor it.

Hmm, still thinking about how we can fit this in, time wise.