Page MenuHomePhabricator

Remove dependency on BytesIO in feature_nulldummy.py
ClosedPublic

Authored by jasonbcox on Feb 9 2019, 00:01.

Details

Summary

Cleanup

Test Plan

test_runner.py feature_nulldummy

Diff Detail

Repository
rABC Bitcoin ABC
Branch
nulldummy-fromhex
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4946
Build 7955: Bitcoin ABC Buildbot (legacy)
Build 7954: arc lint + arc unit

Event Timeline

Fabien requested changes to this revision.Feb 9 2019, 09:05
Fabien added inline comments.
test/functional/feature_nulldummy.py
8 ↗(On Diff #7260)

Please import from messages.py. It works with mininode because it imports all from messages.py for compatibility reason.

92 ↗(On Diff #7260)

You can avoid the unused variable and return directly.

This revision now requires changes to proceed.Feb 9 2019, 09:05
jasonbcox marked 2 inline comments as done.

Fix according to feedback

This revision is now accepted and ready to land.Feb 10 2019, 09:39
This revision was automatically updated to reflect the committed changes.