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

2022-11 CWG Motion 9 #5953

Merged
merged 1 commit into from Nov 24, 2022
Merged

2022-11 CWG Motion 9 #5953

merged 1 commit into from Nov 24, 2022

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Nov 16, 2022

P2564R3 consteval needs to propagate up

Fixes #5964.
Fixes cplusplus/nbballot#467
Fixes cplusplus/papers#1223

Comment on lines +7841 to +7867
\indexdefn{expression!immediate-escalating}%
\indexdefn{conversion!immediate-escalating}%
\indexdefn{immediate-escalating!expression|see{expression, immediate-escalating}}%
\indexdefn{immediate-escalating!conversion|see{conversion, immediate-escalating}}%
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jensmaurer, @Eelis, @JohelEGP: Would you know if there's a better way to index all these cases and permutations?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note also the index entry for "immediately-escalating function" below.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nothing from the top of my mind.

I usually look at macros.tex to see if anything does a better job. Another method I use is looking for existing wording to see how it's done in hopes of finding a better formulation.

Copy link
Member

Choose a reason for hiding this comment

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

@tkoeppe

\defnadj{immediate-escalating}{expression}

should be your one-stop shop here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jensmaurer Not quite, because that also prints output, right?

Copy link
Contributor

@JohelEGP JohelEGP Nov 16, 2022

Choose a reason for hiding this comment

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

Using \defnadj requires tweaking the wording. It also undoes the nesting of the adjective in the index:
1668617866
One option I see is defining \indexadj, which does the indexing part of \defnadj. But that would also undo the above.

Minor editorial changes:
* omitted colons before sentence-lists
* marked "constexpr" and "consteval" specifiers up as keywords
* use "class", not "typename", in templates in the example

Fixes NB DE 046 (C++23 CD).
@tkoeppe tkoeppe merged commit fa07eb4 into main Nov 24, 2022
@tkoeppe tkoeppe deleted the motions-2022-11-cwg-9 branch May 10, 2023 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants