[electrum] minor cleanup in Transaction.verify_signature
ClosedPublic
Actions

Authored by PiRK on Thu, Jun 6, 09:10.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC7abe30a0a8b1: [electrum] minor cleanup in Transaction.verify_signature

Summary

Remove an unnecessary level of indentation.
Merge two except blocks that do the same thing, which allows removing ecdsa includes in this module (we don't need to know about such implementation details in this scope).

Depends on D16301

Test Plan

python test_runner.py

Diff Detail

Repository

rABC Bitcoin ABC

Branch

new_bitcoin_app

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 29246
Build 58028: Build Diff	electrum-tests
Build 58027: arc lint + arc unit

Event Timeline

PiRK created this revision.Thu, Jun 6, 09:10

Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Jun 6, 09:10

PiRK requested review of this revision.Thu, Jun 6, 09:10

Harbormaster completed remote builds in B29246: Diff 48145.Thu, Jun 6, 09:12

Fabien accepted this revision.Thu, Jun 6, 09:49

This revision is now accepted and ready to land.Thu, Jun 6, 09:49

rebase

Harbormaster completed remote builds in B29361: Diff 48332.Thu, Jun 20, 12:27

Closed by commit rABC7abe30a0a8b1: [electrum] minor cleanup in Transaction.verify_signature (authored by PiRK). · Explain WhyThu, Jun 20, 15:26

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC7abe30a0a8b1: [electrum] minor cleanup in Transaction.verify_signature.

Revision Contents
Changeset List

Path

Size

electrum/

electrumabc/

transaction.py

41 lines

Diff 48145

View Options

electrum/electrumabc/transaction.py

[electrum] minor cleanup in Transaction.verify_signatureClosedPublicActions