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

[structure.summary] Remove obsolete paragraph about Note(s): and Example(s): elements that we no longer provide #1825

Merged
merged 2 commits into from Nov 22, 2017

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Nov 19, 2017

No description provided.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Nov 19, 2017

Note(s) removed in 23d7399.

An "Example(s)" element never seems to have existed, at least there was never a Fundesc macro for it.

@jensmaurer
Copy link
Member

jensmaurer commented Nov 20, 2017

What about the "Note:" in table 91 (unordered associative containers), row for b.cbegin()? It's not using the macro, but it's pretty close. Other than that, your change seems to drop the "paragraphs are normative" statement. Do we think that's implied by being an ISO standard?

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Nov 20, 2017

@jensmaurer: That should just be converted to a normal [Note:.

@tkoeppe tkoeppe force-pushed the delement branch 2 times, most recently from 46bbf45 to 196a6ca Compare November 21, 2017 21:29
\begin{note}
\tcode{[b.cbegin(n), b.cend(n))} is a valid range containing
all of the elements in the $\texttt{n}^{\textrm{ th}}$ bucket.
\end{note}%
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where (if anywhere) is the normative rule justifying this claim? Should we copy the corresponding wording from b.begin(n) and b.end(n) to b.cbegin(n) and b.cend(n), or is there general wording saying how begin and cbegin relate somewhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For ordinary iterators and begin/end, you could squint at Table 83 to conclude this relation. But we never have local iterators anywhere. We don't have equivalent "operational semantics" fields in this table.

It might be safest to repeat the wording from the non-const versions and delete the note. What do you think?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That approach sounds good to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, done, PTAL.

…ple(s): elements that we no longer provide

Also restyle a manual Note: in [unord.req] to use the standard note environment.
@zygoloid zygoloid merged commit b3b8a19 into cplusplus:master Nov 22, 2017
@tkoeppe tkoeppe deleted the delement branch November 22, 2017 21:51
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

3 participants