Proxy support for find algorithms
The find family of algorithms does not support proxy reference types, as it expects dereferencing an iterator yields an lvalue here:
https://github.com/STEllAR-GROUP/hpx/blob/dcf267d7868a0deab2e6f746ca5be8dad6553eea/libs/core/algorithms/include/hpx/parallel/algorithms/detail/find.hpp#L50
and a few other similar places.
The fix is to perfectly forward v.
Excellent catch!
Hi, I believe this issue was resolved by PR #6753 Please take a look.
Hi @hkaiser, The PR I mentioned does not add any new tests for this change. I wasn’t sure about the best way to write a test for this case. Could you please guide me on how to add one, or point me to an existing test that I can model it after?
Would libs/core/algorithms/tests/unit/algorithms/find.cpp be a good reference to look at?
The easiest would probably be a test that involves iterators provided from std::vector<bool>.
Currently working on the PR, I’ll update you soon