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.

Sign up for GitHub

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

Jump to bottom

US 39-099 23.3.6.4 [indirectcallable.indirectinvocable] Incorrectly constrained projections #516

Closed
wg21bot opened this issue Oct 31, 2022 · 6 comments · Fixed by cplusplus/draft#6137
Closed

US 39-099 23.3.6.4 [indirectcallable.indirectinvocable] Incorrectly constrained projections #516

wg21bot opened this issue Oct 31, 2022 · 6 comments · Fixed by cplusplus/draft#6137
Labels
accepted LWG Library lwg-ready ranges std::ranges
Milestone
CD C++23

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Oct 31, 2022

Range algorithms which take projections are improperly constrained on the combination of callable/value/reference/projection types, rejecting valid code.

Proposed change:

Adopt P2609.
@wg21bot wg21bot added the LWG Library label Oct 31, 2022
@jensmaurer jensmaurer changed the title US 23.3.6.4 [indirectcallable.indirectinvocable] Incorrectly constrained projections US 39-099 23.3.6.4 [indirectcallable.indirectinvocable] Incorrectly constrained projections Nov 3, 2022
@jensmaurer jensmaurer transferred this issue from another repository Nov 3, 2022
@jensmaurer jensmaurer added this to the CD C++23 milestone Nov 3, 2022
@JeffGarland
Copy link
Member

LWG already saw this paper in Kona and it needs revision before adoption.

cplusplus/papers#1271

@jensmaurer
Copy link
Member

LEWG said in cplusplus/papers#1271 "for C++26"; doesn't that mean the NB comment (which applies to C++23) is rejected?

@timsong-cpp
Copy link

@JeffGarland There's a new revision at https://isocpp.org/files/papers/P2609R2.html

@jensmaurer I don't believe LEWG ever discussed the NB comment.

@JeffGarland
Copy link
Member

Thanks @timsong-cpp I missed the revision - adding to the Issaquah schedule.

@inbal2l inbal2l added the ranges std::ranges label Feb 7, 2023
@JeffGarland
Copy link
Member

Reviewed and approved at Issaquah -- see paper issue for details.

@jensmaurer jensmaurer mentioned this issue Feb 18, 2023
Merged
@jwakely
Copy link
Member

jwakely commented Mar 23, 2023

Accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accepted LWG Library lwg-ready ranges std::ranges
Projects
2023-02 Issaquah LWG Schedule
Status: Ready
Milestone
CD C++23
Development

Successfully merging a pull request may close this issue.

[LWG 8] P2609R3 Relaxing Ranges Just A Smidge cplusplus/draft
6 participants
@jwakely @JeffGarland @timsong-cpp @inbal2l @jensmaurer @wg21bot