[electrum] minor cleanup in Transaction.verify_signature
ClosedPublic
Actions

Authored by PiRK on Jun 6 2024, 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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 6 2024, 09:10

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

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

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

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

rebase

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

Closed by commit rABC7abe30a0a8b1: [electrum] minor cleanup in Transaction.verify_signature (authored by PiRK). · Explain WhyJun 20 2024, 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 48333

View Options

electrum/electrumabc/transaction.py

[electrum] minor cleanup in Transaction.verify_signatureClosedPublicActions