-
-
Notifications
You must be signed in to change notification settings - Fork 31k
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
Allow copying of CodecInfo objects #66143
Comments
CodecInfo objects as retrieved from codecs.lookup currently throw an exception when trying to copy or pickle them. I have attached a patch with a fix and tests. |
The change to codecs.py seems simple enough but would we usually use plain asserts in test code? |
@lehmannro Are you gonna continue this task? |
The PR is ready for review. @serhiy-storchaka Can you review it? |
Co-authored-by: Robert Lehmann <[email protected]>
Thanks @lehmannro for the initial patch and @furkanonder for bringing the issue to a successful end. |
…nGH-109235) Co-authored-by: Robert Lehmann <[email protected]>
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
Linked PRs
The text was updated successfully, but these errors were encountered: