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

[iterator.concept.readable] remove obsolete Note #5408

Merged
merged 1 commit into from Apr 26, 2022

Conversation

huixie90
Copy link
Contributor

There is a note in this section

The expression *i is indirectly required to be valid via the exposition-only dereferenceable concept

This makes sense before P1878R1. But after P1878R1, the expression *i
is "directly" required in the requires clause. So this note is no longer
valid.

The change that adds *i in the require clause is here
333b6eb#diff-0043c904d3e034a77b44feeddf18719e63e646d55db1c564f2a5defe7330c9eaR1213

There is a note in this section
> The expression *i is indirectly required to be valid via the exposition-only dereferenceable concept

This makes sense before P1878R1. But after P1878R1, the expression `*i`
is "directly" required in the requires clause. So this note is no longer
valid.
@jensmaurer
Copy link
Member

@CaseyCarter , @jwakely , would you agree?

@CaseyCarter
Copy link
Contributor

@CaseyCarter , @jwakely , would you agree?

Absolutely. This note is just confusing now.

@jensmaurer jensmaurer merged commit b075835 into cplusplus:main Apr 26, 2022
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.

None yet

4 participants