Page MenuHomePhabricator
Differential D2354

Renamed TestBuilder.Push( string hex ) to .PushHex( string hex )
AbandonedPublic
Actions

Authored by nakihito on Jan 20 2019, 23:05.

Details

Reviewers
jasonbcox
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

Begins work on T393. Renamed TestBuilder's Push( string hex ) to PushHex( string hex ).

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
CategorizingPushes
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4598
Build 7259: Bitcoin ABC Buildbot (legacy)
Build 7258: arc lint + arc unit

Event Timeline

nakihito created this revision.Jan 20 2019, 23:05
Owners added a reviewer: Restricted Owners Package.Jan 20 2019, 23:05
Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 20 2019, 23:05
Herald added a subscriber: schancel. · View Herald Transcript
teamcity added a subscriber: teamcity.Jan 20 2019, 23:05

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

deadalnix added a comment.Jan 20 2019, 23:11

I don't understand what this task is about. What is inconsistent ? Why is this naming more consistent ? Who's going to maintain that test going forward ?

teamcity added a comment.Jan 20 2019, 23:14

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4598: Diff 6767.Jan 20 2019, 23:17
nakihito added a comment.Jan 20 2019, 23:34

My understanding is that the ticket is addressing a code readability issue. You want it to be easier to identify what version of Push() is being used by renaming the overloaded functions based on the argument the function takes. However, I've just been informed that there is an issue of code ownership with these types of changes so I'll close this diff.

nakihito abandoned this revision.Jan 20 2019, 23:34

Revision Contents
Changeset List

PathSize
src/
test/
6 lines

Diff 6767

View Options

src/test/script_tests.cpp

Loading...