Page MenuHomePhabricator
Differential D12787

Rename sigOps to sigCheks in the policy files
ClosedPublic
Actions

Authored by Fabien on Dec 7 2022, 13:56.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCe48f0a5ad3b0: Rename sigOps to sigCheks in the policy files
Summary

Similar to D12785, but for the policy variables/functions.

Depends on D12785 and D12786.

Test Plan
ninja all check-all
ninja bitcoin-fuzzers

Diff Detail

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

Event Timeline

Fabien created this revision.Dec 7 2022, 13:56
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 7 2022, 13:56
Fabien requested review of this revision.Dec 7 2022, 13:56
Harbormaster completed remote builds in B21418: Diff 37050.Dec 7 2022, 14:18
PiRK accepted this revision.Dec 7 2022, 14:27
PiRK added a subscriber: PiRK.
PiRK added inline comments.
src/init.cpp
1994 ↗(On Diff #37050)

shouldn't we also rename the "-bytespersigop" argument ?

This revision is now accepted and ready to land.Dec 7 2022, 14:27
Fabien added inline comments.Dec 7 2022, 14:29
src/init.cpp
1994 ↗(On Diff #37050)

Yes, and also some RPC. I will do that in separated diffs

Fabien added a child revision: D12790: Rename the remaining sigOps to sigChecks.Dec 7 2022, 14:45
Closed by commit rABCe48f0a5ad3b0: Rename sigOps to sigCheks in the policy files (authored by Fabien). · Explain WhyDec 7 2022, 16:26
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCe48f0a5ad3b0: Rename sigOps to sigCheks in the policy files.

Revision Contents
Changeset List

PathSize
src/
4 lines
policy/
16 lines
18 lines
6 lines
2 lines
test/
fuzz/
5 lines

Diff 37064

View Options

src/init.cpp

Loading...
View Options

src/policy/policy.h

Loading...
View Options

src/policy/policy.cpp

Loading...
View Options

src/policy/settings.h

Loading...
View Options

src/policy/settings.cpp

Loading...
View Options

src/test/fuzz/integer.cpp

Loading...