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

[expr.const] Add missing paragraph break #4056

Closed
wants to merge 1 commit into from

Conversation

JohelEGP
Copy link
Contributor

No description provided.

@jensmaurer
Copy link
Member

I'm not convinced we should have a \pnum here. We use "permitted result of a constant expression" right before the bulleted list here.

@JohelEGP
Copy link
Contributor Author

I agree, because there's other places in [expr.const] that could similarly have a \pnum added.

My intention was to make grammatical paragraphs map to paragraph number. I vaguely remember seeing this kind of inconsistency elsewhere, but if the editors agree to this mapping, I could make a more general PR for this subclause. Otherwise, this should be closed.

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Jun 24, 2020
@zygoloid
Copy link
Member

I think this is intentionally structured as it is: this is all part of the same definition of "constant expression", which uses an auxiliary term recursively defined in terms of itself. Keeping this all in the same paragraph seems important -- if nothing else -- to identify the referent of "the above constraints".

@zygoloid zygoloid closed this Sep 18, 2020
@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Sep 18, 2020
@JohelEGP JohelEGP deleted the expr.const branch September 21, 2020 22:35
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