Page MenuHomePhabricator

Fix spelling of "unreachable" in OP_CHECKMULTISIG
ClosedPublic

Authored by tobias_ruck on Sep 15 2021, 23:05.

Details

Reviewers
PiRK
Mengerian
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCe3c96f8ddab2: Fix spelling of "unreachable" in OP_CHECKMULTISIG
Summary

Wrong spelling of "unreachable" in the implementation of OP_CHECKMULTISIG/OP_CHECKMULTISIGVERIFY.

Test Plan

ninja check

Diff Detail

Event Timeline

Owners added a reviewer: Restricted Owners Package.Sep 15 2021, 23:05
PiRK requested changes to this revision.Sep 16 2021, 04:43
PiRK added a subscriber: PiRK.

Can you also remove the extra e after the h, while you are at it.

This revision now requires changes to proceed.Sep 16 2021, 04:43

Actually fix the typo in "unreachable" in OP_CHECKMULTISIG

This revision is now accepted and ready to land.Sep 16 2021, 08:57

It was wrong in multiple ways :)

Mengerian added a subscriber: Mengerian.

LGTM.

I also checked that this typo isn't in Bitcoin Core, it originated with ABC here: https://reviews.bitcoinabc.org/D3474