Page MenuHomePhabricator

Fix script error description
ClosedPublic

Authored by florian on Feb 6 2019, 04:11.

Details

Reviewers
deadalnix
Mengerian
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGINGb1eb229c4c3f: Fix script error description
rABCb1eb229c4c3f: Fix script error description
Summary

As per title

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a reviewer: Restricted Owners Package.Feb 6 2019, 04:11
florian retitled this revision from Fixed script error description to Fix script error description.Feb 6 2019, 04:26
Mengerian requested changes to this revision.Feb 6 2019, 05:03
Mengerian added a subscriber: Mengerian.

Good catch! I indicated a minor suggested rewording

src/script/script_error.cpp
75 ↗(On Diff #7194)

Seems to me it could be better as: "Only push operators allowed in scriptSig"

More accurate that way.

Also in the context of the other messages, which use "signature" to refer just the ECDSA signature.

This revision now requires changes to proceed.Feb 6 2019, 05:03
florian marked an inline comment as done.

signtures->scriptSig

src/script/script_error.cpp
75 ↗(On Diff #7198)

signature scripts ? scriptSig is really a bad name.

florian marked an inline comment as done.

scriptSig -> signature scripts

Sure, "signature scripts" works

This revision is now accepted and ready to land.Feb 6 2019, 15:14
This revision was automatically updated to reflect the committed changes.