Skip to content

Conversation

chgnrdv
Copy link
Contributor

@chgnrdv chgnrdv commented Jun 16, 2023

Fixes #105873

  • changed format specifier in PyUnicode_FromFormat call to accept raw string, pass tp_name of exception as second argument
  • fixed test

…d exception

* changed format specifier in `PyUnicode_FromFormat` call to accept raw string, pass `tp_name` of exception as second argument
* fixed test
Copy link
Member

@ericsnowcurrently ericsnowcurrently left a comment

Choose a reason for hiding this comment

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

LGTM

@brettcannon
Copy link
Member

@ericsnowcurrently any reason you chose to hold off on merging?

@ericsnowcurrently
Copy link
Member

I'm mostly unavailable for the next couple weeks.

@brettcannon brettcannon removed their request for review July 7, 2023 22:23
@kumaraditya303 kumaraditya303 enabled auto-merge (squash) July 8, 2023 08:17
@kumaraditya303 kumaraditya303 merged commit 69a39bd into python:main Jul 8, 2023
@ericsnowcurrently
Copy link
Member

Thanks @kumaraditya303!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

_xxsubinterpreters: the name of shared exception type is str(exc_type), not exc_type.__name__
5 participants