Skip to content

Implement the "binary search" ranges algorithm family - #917

Merged
Casey Carter (CaseyCarter) merged 5 commits into
microsoft:masterfrom
CaseyCarter:bsearch
Jun 30, 2020
Merged

Casey Carter (CaseyCarter) merged 5 commits into
microsoft:masterfrom
CaseyCarter:bsearch

Conversation

@CaseyCarter

@CaseyCarter Casey Carter (CaseyCarter) commented Jun 24, 2020

Copy link
Copy Markdown
Contributor

Includes ranges::lower_bound, ranges::upper_bound, ranges::equal_range, and ranges::binary_search.

Partially addresses #39.

Includes `ranges::lower_bound`, `ranges::upper_bound`, `ranges::equal_range`, and `ranges::binary_search`.
@CaseyCarter Casey Carter (CaseyCarter) added the cxx20 C++20 feature label Jun 24, 2020
@CaseyCarter
Casey Carter (CaseyCarter) requested a review from a team as a code owner June 24, 2020 14:55
@CaseyCarter Casey Carter (CaseyCarter) changed the title Implement the "binary search" algorithm family: Implement the "binary search" ranges algorithm family: Jun 24, 2020

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minimal comment regarding _Ubegin but otherwise this looks great.

Comment thread stl/inc/algorithm Outdated
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) changed the title Implement the "binary search" ranges algorithm family: Implement the "binary search" ranges algorithm family Jun 25, 2020
Comment thread stl/inc/algorithm Outdated
Comment thread stl/inc/algorithm
Comment thread stl/inc/algorithm Outdated
Comment thread stl/inc/algorithm Outdated
@cbezault Curtis J Bezault (cbezault) removed their assignment Jun 29, 2020
@CaseyCarter
Casey Carter (CaseyCarter) merged commit ac5c8b0 into microsoft:master Jun 30, 2020
@CaseyCarter
Casey Carter (CaseyCarter) deleted the bsearch branch June 30, 2020 05:45
@CaseyCarter Casey Carter (CaseyCarter) removed their assignment Jun 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cxx20 C++20 feature ranges C++20/23 ranges

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants