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

[range.range] Redundant paragraph about expression variations LWG3915 #4431

Open
JohelEGP opened this issue Dec 24, 2020 · 4 comments
Open

[range.range] Redundant paragraph about expression variations LWG3915 #4431

JohelEGP opened this issue Dec 24, 2020 · 4 comments
Labels
lwg Issue must be reviewed by LWG.

Comments

@JohelEGP
Copy link
Contributor

Expression variations kick in for "an expression that is non-modifying for some constant lvalue operand", but std::ranges::range's is an non-constant lvalue, so p2 is redundant.
@JohelEGP
Copy link
Contributor Author

I suppose that the change that clarified the template parameters' cv-qualification for purposes of equality-preservation and requiring additional variations happened concurrently with the change of std::ranges::range's operand from a forwarding reference to a non-constant lvalue reference.

@jensmaurer jensmaurer added the lwg Issue must be reviewed by LWG. label Jan 3, 2021
@jensmaurer
Copy link
Member

@jwakely , do you feel we can editorially remove p2, or should this get LWG review?

@JohelEGP
Copy link
Contributor Author

JohelEGP commented Apr 1, 2023

Requested for an issue to be opened.

@Dani-Hub
Copy link
Member

Dani-Hub commented Apr 2, 2023

@JohelEGP A new library has been created, see LWG 3915.

@JohelEGP JohelEGP changed the title [range.range] Redudant paragraph about expression variations [range.range] Redudant paragraph about expression variations LWG3915 Apr 2, 2023
@JohelEGP JohelEGP changed the title [range.range] Redudant paragraph about expression variations LWG3915 [range.range] Redundant paragraph about expression variations LWG3915 Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lwg Issue must be reviewed by LWG.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Dani-Hub @JohelEGP @jensmaurer