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
[range.zip, ranges.cartesian.iterator] Simplify maybe-const<Const, Views>
into const Views
#5778
Conversation
…ews> into const Views
Is this editorial? |
Not sure, if this was LWG then I would close it. |
Well, to be editorial the change cannot change the normative meaning of the text, first and foremost. I don't know whether the proposed new text is equivalent to the existing one, or if you're proposing a (simplifying) change of the observable specification. Can you elaborate? |
IIUC the idea is: these |
I believe this has no normative impact. The requires clause will short circuit so that if So I think this is a valid editorial simplification. Edit: i.e. what cpplearner said at the same time. |
OK, thanks! In that case, can you please review if any of the linebreaks can now go away? |
Elsewhere it is simple enough since there is no fold expression and
Base
is defined, and I think this improves readability.