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

[except.handle] group all paragraphs on searching for a handler #7283

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

AlisdairM
Copy link
Contributor

This commit moves all of the paragraphs involved in the search for a handler for an exception into a single logical sequence.

After this change, [except.spec] deals only with specifying the noexcept function decorator and its interaction with the noexcept operator, and contains no text regarding exceptions themselves. It might be appropriate to move that subclause into the [dcl] structure at a future date.

This commit moves all of the paragraphs involved in the search for a
handler for an exception into a single logical sequence.

After this change, [except.spec] deals only with specifying the
'noexcept' function decorator and its interaction with the
'noexcept' operator, and contains no text regarding exceptions
themselves.  It might be appropriate to move that subclause into
the [dcl] structure at a future date.
@tkoeppe
Copy link
Contributor

tkoeppe commented Oct 16, 2024

@jensmaurer Any thoughts on whether this improves the order of presentation?

@AlisdairM
Copy link
Contributor Author

I have just pushed #7308 which is the main change I wanted to make for exceptions in the clause reorganization. I have pushed it as a follow-up, based on this PR.

Understood it is late to be bringing this to the table --- that is why I tried to ping the edit list early :)

@tkoeppe
Copy link
Contributor

tkoeppe commented Oct 16, 2024

Exceptions aren't part of the planned clause reorganizations (#5226), though, so no worries. This smaller cleanup is independent.

Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

This feels like an agreeable local text move to me.

@tkoeppe tkoeppe merged commit 198e991 into cplusplus:main Oct 16, 2024
2 checks passed
@AlisdairM AlisdairM deleted the collect_all_handling branch October 16, 2024 19:54
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 this pull request may close these issues.

3 participants