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

P2036R3 Change scope of lambda trailing-return-type #5003

Merged
merged 1 commit into from Oct 16, 2021

Conversation

jensmaurer
Copy link
Member

@jensmaurer jensmaurer added this to the post-2021-10 milestone Oct 5, 2021
source/expressions.tex Outdated Show resolved Hide resolved
source/expressions.tex Outdated Show resolved Hide resolved
source/expressions.tex Outdated Show resolved Hide resolved
source/expressions.tex Show resolved Hide resolved
@JohelEGP
Copy link
Contributor

JohelEGP commented Oct 9, 2021

Looks like I misunderstood \keyword. I have removed those suggestions here and in my other reviews.

@jensmaurer
Copy link
Member Author

Looks like I misunderstood \keyword. I have removed those suggestions here and in my other reviews.

I think \keyword would be applicable in some of these places. No big deal if it doesn't happen, though.

@JohelEGP
Copy link
Contributor

My understanding is that, since \keyword adds index entries, it should only be used when the meaning of the keyword is defined. That is unlike \libconcept, which is meant to index all uses of concepts.

In that case, changing all "predicate is \tcode{true}" to use \keyword would have the undesirable effect of adding many index entries.

@jensmaurer
Copy link
Member Author

In the core language sections, I sprinkled \keyword all around when mentioned outside of (example) codeblocks. (Presumably, the core language only talks about a keyword when that has a meaning in that situation.)

We probably shouldn't do that for the library.

source/expressions.tex Outdated Show resolved Hide resolved
@tkoeppe tkoeppe merged commit 743fcd7 into main Oct 16, 2021
@jensmaurer jensmaurer deleted the motions-2021-10-cwg-8 branch February 9, 2022 09:10
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