-
Notifications
You must be signed in to change notification settings - Fork 779
[Motions 2025 06 lwg 1] P3742R0 C++ Standard Library Issues #8014
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some \linebreak
s introduced which should be \brk{}
; otherwise I couldn't find anything wrong.
There are also some index entries added for exposition-only stuff, and I'm not sure if that's usually done.
As I understand it, |
Yeah, with the exception that it won't leave a hyphen at line wraps (unless
Can you just replace any use of
Jens also replaced a bunch of |
…ying results throws
…er itself instead of its child [exec.let] Added : before codeblock for consistency.
… expression is only evaluated once
[expected.object.cons]p23.2 Fly-by fix expression (">" in wrong place).
…tinel_t should be noexcept
9d78e1e
to
54ce6ab
Compare
I will do this next time. Since this is a multi-commit motion, making those changes will add more work for Thomas. That said, I have come across many cases where \brk did not work and I had to resort to \linebreak (perhaps it has happened so often that I use \linebreak more often than I should - ok, will always check in the future). |
There is a way to squash the I am not convinced The rest of the PR looks good. |
@@ -2577,6 +2577,9 @@ | |||
\item | |||
Otherwise, \tcode{\exposid{connect-awaitable}(new_sndr, rcvr)}. | |||
\end{itemize} | |||
%%FIXME: This sentence should be joined with the previous sentences; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this looks very odd, but that's what it says in the paper. We should follow this up and reorganize this.
Fixes #7938