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

[basic.fundamental] Itemize p13 #6464

Closed

Conversation

Eisenwave
Copy link
Contributor

image

This is a very slight readability improvement which highlights the already existing symmetry in the paragraph between what is said for the two infinities.

Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

Hm... I'm not convinced this is a net improvement. I can certainly live with the result.
Bulleted lists in the standard are usually used for conjunctions or disjunctions or for lists of conditions/rules (expr.const or basic.def.odr). This case doesn't feel like any of those.

@tkoeppe, what do you think?

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 20, 2023

I'm inclined to agree that this isn't really improving anything. The status quo seems entirely adequate. One could easily imagine an opposite desire to "declutter the presentation by turning trivial lists into fluent sentences", say, and we could endlessly churn on something like that. So I'd say we should only consider such rewrites if there is a very clear and unambiguous improvement. Having an underlying principle is great, too (such as "conjunctions and disjunctions can benefit from list style"), but I'd not want to suggest some absolute rule here. (The best way to present something clearly often depends on a lot of factors, and there should be room for discretion.)

@JohelEGP
Copy link
Contributor

With regards to clear presentation, I'm glad lists are more welcome now.
They're a powerful tool for the job.

@jensmaurer
Copy link
Member

Let's bury this, then.

@jensmaurer jensmaurer closed this Aug 20, 2023
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