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

[over.best.ics] Minor reading improvements #5023

Closed
dangelog opened this issue Oct 12, 2021 · 2 comments · Fixed by #5086
Closed

[over.best.ics] Minor reading improvements #5023

dangelog opened this issue Oct 12, 2021 · 2 comments · Fixed by #5086
Assignees

Comments

@dangelog
Copy link
Contributor

The end of https://eel.is/c++draft/over.best.ics#over.ics.list-8 says:

[...] the implicit conversion sequence is a user-defined conversion sequence with the second standard conversion sequence an identity conversion.

Could it be possible to rephrase this sentence like this:

[...] the implicit conversion sequence is a user-defined conversion sequence with the second standard conversion sequence being an identity conversion.

?

I personally think it would bring more clarity, and be less hostile to be parsed by human beings -- there's no verb apart from "is". In another related paragraph, an extra verb is actually already there: https://eel.is/c++draft/over.best.ics#over.ics.list-2.sentence-1

in which case the implicit conversion sequence is a user-defined conversion sequence whose second standard conversion sequence is an identity conversion.

The same kind of change (adding a second verb such as "being", or using a relative sentence) could also be applied in another couple of places:

https://eel.is/c++draft/over.best.ics#over.ics.ref-1.sentence-2
https://eel.is/c++draft/over.best.ics#over.ics.list-7.2.sentence-1

@jensmaurer
Copy link
Member

I like "whose second standard conversion sequence is ..." instead of adding "being".

@dangelog
Copy link
Contributor Author

dangelog commented Nov 4, 2021

Sure, that sounds fine by me.

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 a pull request may close this issue.

2 participants