Page MenuHomePhabricator

refactor: replace qLowerBound & qUpperBound with std:: upper_bound & lower_bound
ClosedPublic

Authored by majcosta on Dec 13 2019, 16:09.

Details

Reviewers
deadalnix
jasonbcox
nakihito
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABC826e21daed54: refactor: replace qLowerBound & qUpperBound with std:: upper_bound & lower_bound
Summary

This is a backport of Core PR16674 - https://github.com/bitcoin/bitcoin/pull/16674

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

majcosta created this revision.Dec 13 2019, 16:09
Owners added a reviewer: Restricted Owners Package.Dec 13 2019, 16:09
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 13 2019, 16:09
majcosta planned changes to this revision.Dec 13 2019, 16:15

going to add the other two commits to this revision

majcosta updated this revision to Diff 14841.Dec 13 2019, 16:35

added commits fea33cbbdfb4673033f3414bf1613591ff654aac and 59373e3e94015316bcaa03a7b9c2e6f442641720

majcosta updated this revision to Diff 14842.Dec 13 2019, 16:40

accidentally removed brackets from single line if statement. corrected.

majcosta edited the summary of this revision. (Show Details)Dec 13 2019, 16:44
jasonbcox requested changes to this revision.Dec 13 2019, 18:02
jasonbcox added a subscriber: jasonbcox.

Please also remove the strikethrough portions of the summary. This is going to end up showing up in git log and I don't see how it will be useful after this is landed.

src/qt/bantablemodel.cpp
62 ↗(On Diff #14842)

add brackets

This revision now requires changes to proceed.Dec 13 2019, 18:02
majcosta edited the summary of this revision. (Show Details)Dec 13 2019, 18:04
majcosta updated this revision to Diff 14844.Dec 13 2019, 18:08

included missing brackets around single line if statement

jasonbcox accepted this revision.Dec 13 2019, 20:12
This revision is now accepted and ready to land.Dec 13 2019, 20:12
nakihito accepted this revision.Dec 13 2019, 21:03