Page MenuHomePhabricator

[QA] Renaming rawtx into rawTx

Authored by jasonbcox on Feb 8 2019, 18:58.



Follow up from #10069 is using two variables: rawtx and rawTx, which can be easily confused (see #10069). This PR rename everything in rawtx.

Inverted backport of Core PR10083 (rawtx -> rawTx instead of rawTx -> rawtx)
Depends on D2522

Test Plan rpc_fundrawtransaction

Diff Detail

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

Event Timeline

jasonbcox created this revision.Feb 8 2019, 18:58
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 8 2019, 18:58
Herald added a subscriber: schancel. · View Herald Transcript
deadalnix requested changes to this revision.Feb 8 2019, 19:01
deadalnix added inline comments.
341 ↗(On Diff #7247)

Now this is inconsistent and probably exactly how the error happened in the first place.

This revision now requires changes to proceed.Feb 8 2019, 19:01
jasonbcox updated this revision to Diff 7251.Feb 8 2019, 20:06

Rebase + invert the change to be rawtx -> rawTx since this naming is more to style

jasonbcox retitled this revision from [QA] Renaming rawTx into rawtx to [QA] Renaming rawtx into rawTx.Feb 8 2019, 20:07
jasonbcox edited the summary of this revision. (Show Details)
deadalnix accepted this revision.Feb 8 2019, 22:38
deadalnix added inline comments.
94 ↗(On Diff #7251)

hahahaha, WTF. Seriously...

This revision is now accepted and ready to land.Feb 8 2019, 22:38
This revision was automatically updated to reflect the committed changes.