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

normalize exception in PyErr::matches and PyErr::is_instance #3313

Merged
merged 2 commits into from
Jul 14, 2023

Conversation

davidhewitt
Copy link
Member

@davidhewitt davidhewitt commented Jul 11, 2023

Split off from #3306

This fixes a case where PyErr::matches and PyErr::is_instance could return results inconsistent with PyErr::get_type due to the first two methods not ensuring the exception was normalised.

@davidhewitt davidhewitt changed the title normalize exception in PyErr::matches and PyErr::get_type normalize exception in PyErr::matches and PyErr::is_instance Jul 13, 2023
@davidhewitt davidhewitt force-pushed the unnormalized-err-type branch from de5644d to 8909bdd Compare July 14, 2023 11:13
@davidhewitt davidhewitt force-pushed the unnormalized-err-type branch from ac22ff6 to eb8c23e Compare July 14, 2023 12:18
@davidhewitt davidhewitt force-pushed the unnormalized-err-type branch from eb8c23e to f39dd52 Compare July 14, 2023 12:20
@davidhewitt
Copy link
Member Author

I've corrected the header and changelog entry, and pushed a separate commit to action the removal of "normalization" as a term from the PyErr docs.

@adamreichold adamreichold added this pull request to the merge queue Jul 14, 2023
Merged via the queue into PyO3:main with commit 562ef0c Jul 14, 2023
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.

2 participants