-
Notifications
You must be signed in to change notification settings - Fork 748
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
[CPP29 REORG] Dissolve [except] #7320
Open
AlisdairM
wants to merge
7
commits into
cplusplus:main
Choose a base branch
from
AlisdairM:cpp29_dissolve_except
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commits on Oct 18, 2024
-
[except.uncaught] Tidy the specification for uncaught exceptions
Several concurrent fixes. First include the normative wording that 'uncaught_exceptions' returns the number of uncaught exceptions *on the current thread*. This wording is present in the core language. Then move the core wording for when an exception is uncaught directly into the text that talks about caught and uncaught exceptions. In the process, turn the reference to into a note, so that there is only one normative specification. Finally, remove [except.uncaught] as it is now empty, and add the missing descriptive sentence to the intro paragraph of [except.special.general]. This happens to produce quite a pleasing page-break, but that is just luck, not design.
Configuration menu - View commit details
-
Copy full SHA for 2dc8074 - Browse repository at this point
Copy the full SHA 2dc8074View commit details -
[except.terminate] Better describe the function
While 'std:terminate' was originally conceived as the way to report failures in the exception handling machinery, it has evolved to become a more general tool for reporting unrecoverable failures in the C++ runtime. This rewording attempts to address that evolving design, and in doing so addresses the outstanding %FIXME% that the current text is not adequately descriptive in the first place.
Configuration menu - View commit details
-
Copy full SHA for 0ba172e - Browse repository at this point
Copy the full SHA 0ba172eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8c718ea - Browse repository at this point
Copy the full SHA 8c718eaView commit details -
[except.terminate] Move the 'terminate' function to [basic.start]
Now that the terminate function is more general than just failures of the exception machinery, it belongs with Start and termination.
Configuration menu - View commit details
-
Copy full SHA for 494fd23 - Browse repository at this point
Copy the full SHA 494fd23View commit details -
Configuration menu - View commit details
-
Copy full SHA for 42d33f1 - Browse repository at this point
Copy the full SHA 42d33f1View commit details -
Configuration menu - View commit details
-
Copy full SHA for f142ecc - Browse repository at this point
Copy the full SHA f142eccView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2af6f24 - Browse repository at this point
Copy the full SHA 2af6f24View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.