Page MenuHomePhabricator

Merge #12996: tests: Remove redundant bytes(…) calls
ClosedPublic

Authored by nakihito on Mon, Aug 12, 18:47.

Details

Reviewers
deadalnix
Fabien
jasonbcox
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCf6969d52800c: Merge #12996: tests: Remove redundant bytes(…) calls
Summary

9db48c5634 tests: Remove redundant bytes² (practicalswift)

Pull request description:

This is a follow-up to #12993. As @jnewbery noted `bytes()` is idempotent.

Tree-SHA512: 0eb25e0c2c46f9abaac30f964c5eb422bece1414c840a717d86794424294cb19d995a6db7c8df2a2f4ec84776b05274a637f2c111738f397051f510e57184752

Backport of Core PR12996
https://github.com/bitcoin/bitcoin/pull/12996/

Test Plan
make check
test_runner.py

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

nakihito created this revision.Mon, Aug 12, 18:47
Owners added a reviewer: Restricted Owners Package.Mon, Aug 12, 18:47
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Aug 12, 18:47
jasonbcox accepted this revision.Tue, Aug 13, 01:16
This revision is now accepted and ready to land.Tue, Aug 13, 01:16