Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add find_min/max/minmax algorithms #113

Merged
merged 2 commits into from
Aug 7, 2023
Merged

Add find_min/max/minmax algorithms #113

merged 2 commits into from
Aug 7, 2023

Conversation

tcbrindle
Copy link
Owner

These do the same as min()/max()/minmax(), but take multipass sequences and return a cursor to the requisite elements rather than an optional value, returning a past-the-end cursor if the input sequence is empty.

Fixes #112

These do the same as `min()`/`max()`/`minmax()`, but take multipass sequences and return a cursor to the requisite elements rather than an optional value, returning a past-the-end cursor if the input sequence is empty.

Fixes #112
@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (051dce9) 97.58% compared to head (e83bdcb) 97.58%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #113   +/-   ##
=======================================
  Coverage   97.58%   97.58%           
=======================================
  Files          66       66           
  Lines        2276     2276           
=======================================
  Hits         2221     2221           
  Misses         55       55           
Files Changed Coverage Δ
include/flux/core/inline_sequence_base.hpp 91.66% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tcbrindle tcbrindle merged commit d579bfa into main Aug 7, 2023
25 checks passed
@tcbrindle tcbrindle deleted the pr/find_min_max branch August 9, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add find_min/max/minmax algorithms
1 participant