Page MenuHomePhabricator

add missing swap to CScriptCheck
ClosedPublic

Authored by markblundeberg on Sat, Feb 8, 09:44.

Details

Reviewers
jasonbcox
Group Reviewers
Restricted Project
Commits
rABC0f9d3a2e8010: add missing swap to CScriptCheck
Summary

My bad, I missed adding this. It doesn't really matter since
the results of CScriptCheck (neither metrics nor error) are
never used in the parallel setting. In future I'd really like
to make error/metrics not be members of this class anyway,
instead to be optional outparams of the operator().

Test Plan

ninja check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

markblundeberg created this revision.Sat, Feb 8, 09:44
Herald added a reviewer: Restricted Project. · View Herald TranscriptSat, Feb 8, 09:44
jasonbcox accepted this revision.Sat, Feb 8, 20:21
This revision is now accepted and ready to land.Sat, Feb 8, 20:21