| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:9f74b43c-5a36-2275-1398-5dd9a47644a9 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E72E:23658B:58683:7BE35:6A4FB68C |
| html-safe-nonce | 1812c29c7042bed036e7b26f5bce271a6d1c162e680945b99f60a859ae058fdf |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNzJFOjIzNjU4Qjo1ODY4Mzo3QkUzNTo2QTRGQjY4QyIsInZpc2l0b3JfaWQiOiI0MjE0OTQwNDkwNzEyMjAwODQ0IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 18203f196fd8ed1874b063dd2a0955e3d5a2d072016e7269b0961f9e3cae8857 |
| hovercard-subject-tag | pull_request:141199654 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/python/cpython/3593/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/e8afa3be5b6a8360b80231b4ea53f808855edf02d1e81cee1c961a2e606146a0/python/cpython/pull/3593 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/e8afa3be5b6a8360b80231b4ea53f808855edf02d1e81cee1c961a2e606146a0/python/cpython/pull/3593 |
| og:image:alt | The concrete PyDict_* API is used to interact with PyInterpreterState.modules in a number of places. This isn't compatible with all dict subclasses, nor with other Mapping implementations. Th... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | ericsnowcurrently |
| hostname | github.com |
| expected-hostname | github.com |
| None | b92d11c0aa4a77d54ef4af1078b6a15fb5a70a215b30c4ecf28889d5a8e656d9 |
| turbo-cache-control | no-cache |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 4b249b445842943ed31549e027f57a8ade9881ed |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://github.com/python/cpython/pull/3593#start-of-content |
|
| https://github.com/ |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F3593 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub Copilot appDirect agents from issue to merge | https://github.com/features/ai/github-app |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| View all resources | https://github.com/resources |
| GitHub SponsorsFund open source developers | https://github.com/open-source/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/open-source/accelerator |
| GitHub Stars | https://stars.github.com |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/enterprise/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F3593 |
|
Sign up
| https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=python%2Fcpython |
| Reload | https://github.com/python/cpython/pull/3593 |
| Reload | https://github.com/python/cpython/pull/3593 |
| Reload | https://github.com/python/cpython/pull/3593 |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
|
python
| https://github.com/python |
| cpython | https://github.com/python/cpython |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
|
Notifications
| https://github.com/login?return_to=%2Fpython%2Fcpython |
|
Fork
34.8k
| https://github.com/login?return_to=%2Fpython%2Fcpython |
|
Star
73.6k
| https://github.com/login?return_to=%2Fpython%2Fcpython |
|
Code
| https://github.com/python/cpython |
|
Issues
5k+
| https://github.com/python/cpython/issues |
|
Pull requests
2.3k
| https://github.com/python/cpython/pulls |
|
Actions
| https://github.com/python/cpython/actions |
|
Projects
| https://github.com/python/cpython/projects |
|
Security and quality
0
| https://github.com/python/cpython/security |
|
Insights
| https://github.com/python/cpython/pulse |
|
Code
| https://github.com/python/cpython |
|
Issues
| https://github.com/python/cpython/issues |
|
Pull requests
| https://github.com/python/cpython/pulls |
|
Actions
| https://github.com/python/cpython/actions |
|
Projects
| https://github.com/python/cpython/projects |
|
Security and quality
| https://github.com/python/cpython/security |
|
Insights
| https://github.com/python/cpython/pulse |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| python:master | https://github.com/python/cpython/tree/master |
| ericsnowcurrently:sys-modules-any-mapping | https://github.com/ericsnowcurrently/cpython/tree/sys-modules-any-mapping |
| Conversation | https://github.com/python/cpython/pull/3593 |
| Commits30 (30) | https://github.com/python/cpython/pull/3593/commits |
| Checks | https://github.com/python/cpython/pull/3593/checks |
| Files changed | https://github.com/python/cpython/pull/3593/files |
| bpo-28411: Support other mappings in PyInterpreterState.modules. | https://github.com/python/cpython/pull/3593#top |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| python:master | https://github.com/python/cpython/tree/master |
| ericsnowcurrently:sys-modules-any-mapping | https://github.com/ericsnowcurrently/cpython/tree/sys-modules-any-mapping |
|
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 14, 2017 | https://github.com/python/cpython/pull/3593#issue-257885852 |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| #1638 | https://github.com/python/cpython/pull/1638 |
| https://bugs.python.org/issue28411 | https://bugs.python.org/issue28411 |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| September 14, 2017 18:20 | https://github.com/python/cpython/pull/3593#commits-pushed-8f3ecfc |
|
| https://github.com/ericsnowcurrently |
| Add _PyImport_GetModule*. | https://github.com/python/cpython/pull/3593/commits/8f3ecfc676a6dacf71665fc54509b33a8f868198 |
| 8f3ecfc | https://github.com/python/cpython/pull/3593/commits/8f3ecfc676a6dacf71665fc54509b33a8f868198 |
|
| https://github.com/ericsnowcurrently |
| Allow sys.modules to be any mapping. | https://github.com/python/cpython/pull/3593/commits/9dc9dedfb821ee6e7b2d5d976619eccc580e381c |
| 9dc9ded | https://github.com/python/cpython/pull/3593/commits/9dc9dedfb821ee6e7b2d5d976619eccc580e381c |
|
| https://github.com/ericsnowcurrently |
| Add _PyImport_SetModule*. | https://github.com/python/cpython/pull/3593/commits/f2a54b1169ad0ff363b438a3c40ece7b45c35446 |
| f2a54b1 | https://github.com/python/cpython/pull/3593/commits/f2a54b1169ad0ff363b438a3c40ece7b45c35446 |
|
| https://github.com/ericsnowcurrently |
| Add PyImport_GetModule(). | https://github.com/python/cpython/pull/3593/commits/502e1387d0c0a036035bf67924a9e87c80ac3d2e |
| 502e138 | https://github.com/python/cpython/pull/3593/commits/502e1387d0c0a036035bf67924a9e87c80ac3d2e |
|
| https://github.com/ericsnowcurrently |
| Decref the module when done. | https://github.com/python/cpython/pull/3593/commits/ec8c5043ef33c56a140aa1272fb85d56ec7e63e6 |
| ec8c504 | https://github.com/python/cpython/pull/3593/commits/ec8c5043ef33c56a140aa1272fb85d56ec7e63e6 |
|
| https://github.com/ericsnowcurrently |
| Fix ref counts. | https://github.com/python/cpython/pull/3593/commits/a1546f6e969afa856c1627c28e11b82025c70486 |
| a1546f6 | https://github.com/python/cpython/pull/3593/commits/a1546f6e969afa856c1627c28e11b82025c70486 |
|
| https://github.com/ericsnowcurrently |
| Look up "new" modules in the given modules dict. | https://github.com/python/cpython/pull/3593/commits/5a5fad1e01dfd680967d602a495e75d16f8aadd3 |
| 5a5fad1 | https://github.com/python/cpython/pull/3593/commits/5a5fad1e01dfd680967d602a495e75d16f8aadd3 |
|
| https://github.com/ericsnowcurrently |
| Fix error checking. | https://github.com/python/cpython/pull/3593/commits/db0a2471a3f79b1e72ebe8111b06e45fd5bb1590 |
| db0a247 | https://github.com/python/cpython/pull/3593/commits/db0a2471a3f79b1e72ebe8111b06e45fd5bb1590 |
|
| https://github.com/ericsnowcurrently |
| Use PyImport_GetModuleDict() in PyImport_GetModule(). | https://github.com/python/cpython/pull/3593/commits/7082b475ff5ccaabb756d9d9f7f9a9cf2d4b83c2 |
| 7082b47 | https://github.com/python/cpython/pull/3593/commits/7082b475ff5ccaabb756d9d9f7f9a9cf2d4b83c2 |
|
| https://github.com/ericsnowcurrently |
| Add a missing incref. | https://github.com/python/cpython/pull/3593/commits/5082c479790a02c44abb1e7d3c688ad32316db20 |
| 5082c47 | https://github.com/python/cpython/pull/3593/commits/5082c479790a02c44abb1e7d3c688ad32316db20 |
|
| https://github.com/ericsnowcurrently |
| Add a Misc/NEWS entry. | https://github.com/python/cpython/pull/3593/commits/bb03872912d62256ce1254a6f6d7d76ecaca9ed8 |
| bb03872 | https://github.com/python/cpython/pull/3593/commits/bb03872912d62256ce1254a6f6d7d76ecaca9ed8 |
|
| https://github.com/ericsnowcurrently |
| Make the docs for PyImport_GetModule() more clear. | https://github.com/python/cpython/pull/3593/commits/2e07296f7853e84b5646a647a30d4882061c0739 |
| 2e07296 | https://github.com/python/cpython/pull/3593/commits/2e07296f7853e84b5646a647a30d4882061c0739 |
|
| https://github.com/ericsnowcurrently |
| Fix style (bracket placement). | https://github.com/python/cpython/pull/3593/commits/46311635ec97a293ebbbbe3287577e024f8ae1a5 |
| 4631163 | https://github.com/python/cpython/pull/3593/commits/46311635ec97a293ebbbbe3287577e024f8ae1a5 |
|
| https://github.com/ericsnowcurrently |
| Drop _PyImport_GetModuleString(). | https://github.com/python/cpython/pull/3593/commits/5b154b3a71998e89c525a6a89186fea9a094f9fe |
| 5b154b3 | https://github.com/python/cpython/pull/3593/commits/5b154b3a71998e89c525a6a89186fea9a094f9fe |
|
| https://github.com/ericsnowcurrently |
| Use PyDict_CheckExact() for sys.modules in fast case. | https://github.com/python/cpython/pull/3593/commits/33174f36d7a097f09b0932b9e5e791793d7cf42c |
| 33174f3 | https://github.com/python/cpython/pull/3593/commits/33174f36d7a097f09b0932b9e5e791793d7cf42c |
|
| https://github.com/ericsnowcurrently |
| Do not use PyMapping_HasKey() with an error set. | https://github.com/python/cpython/pull/3593/commits/e50e258496d2dc7c312dea49c7b4ba92020e05b6 |
| e50e258 | https://github.com/python/cpython/pull/3593/commits/e50e258496d2dc7c312dea49c7b4ba92020e05b6 |
|
| https://github.com/ericsnowcurrently |
| Revert a code order change. | https://github.com/python/cpython/pull/3593/commits/23cf4934e57691a92af7dcc52e722516425758f3 |
| 23cf493 | https://github.com/python/cpython/pull/3593/commits/23cf4934e57691a92af7dcc52e722516425758f3 |
|
| https://github.com/ericsnowcurrently |
| Switch the code order back. | https://github.com/python/cpython/pull/3593/commits/eed5e24d122263c46b0dbafb59db66bd07e0d966 |
| eed5e24 | https://github.com/python/cpython/pull/3593/commits/eed5e24d122263c46b0dbafb59db66bd07e0d966 |
|
| https://github.com/ericsnowcurrently |
| Only use PyDict_* for dicts. | https://github.com/python/cpython/pull/3593/commits/bcc72a539c695a34501a581d72df9749db97ffe8 |
| bcc72a5 | https://github.com/python/cpython/pull/3593/commits/bcc72a539c695a34501a581d72df9749db97ffe8 |
|
| https://github.com/ericsnowcurrently |
| Add a reference while cleaning up. | https://github.com/python/cpython/pull/3593/commits/981dbd2cbb8495c9e0d49aa908800b2cd50f6183 |
| 981dbd2 | https://github.com/python/cpython/pull/3593/commits/981dbd2cbb8495c9e0d49aa908800b2cd50f6183 |
|
| https://github.com/ericsnowcurrently |
| Fix _pickle. | https://github.com/python/cpython/pull/3593/commits/0fe1aff2874f5c2efe59dece058051ceb100265e |
| 0fe1aff | https://github.com/python/cpython/pull/3593/commits/0fe1aff2874f5c2efe59dece058051ceb100265e |
|
| https://github.com/ericsnowcurrently |
| Fix the NEWS entry. | https://github.com/python/cpython/pull/3593/commits/d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
| d4d9219 | https://github.com/python/cpython/pull/3593/commits/d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| September 14, 2017 23:32 | https://github.com/python/cpython/pull/3593#event-1250194058 |
| https://github.com/bedevere-bot |
| bedevere-bot | https://github.com/bedevere-bot |
|
awaiting merge
| https://github.com/python/cpython/issues?q=state%3Aopen%20label%3A%22awaiting%20merge%22 |
| Sep 14, 2017 | https://github.com/python/cpython/pull/3593#event-1250194093 |
| https://github.com/the-knights-who-say-ni |
| the-knights-who-say-ni | https://github.com/the-knights-who-say-ni |
|
CLA signed
| https://github.com/python/cpython/issues?q=state%3Aopen%20label%3A%22CLA%20signed%22 |
| Sep 14, 2017 | https://github.com/python/cpython/pull/3593#event-1250194107 |
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| force-pushed | https://github.com/python/cpython/compare/5a0b5900e01fc2c0c7476b689751b9c0babeccdb..d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
| 5a0b590 | https://github.com/python/cpython/commit/5a0b5900e01fc2c0c7476b689751b9c0babeccdb |
| d4d9219 | https://github.com/python/cpython/commit/d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
|
Compare
| https://github.com/python/cpython/compare/5a0b5900e01fc2c0c7476b689751b9c0babeccdb..d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
| September 14, 2017 23:33 | https://github.com/python/cpython/pull/3593#event-1250194623 |
| https://github.com/ncoghlan |
| ncoghlan | https://github.com/ncoghlan |
|
Sep 15, 2017
| https://github.com/python/cpython/pull/3593#pullrequestreview-62927359 |
|
View reviewed changes
| https://github.com/python/cpython/pull/3593/files/d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
| Python/import.c | https://github.com/python/cpython/pull/3593/files/d4d9219d59069af71b6d01d7d8ea32443cc922d8#diff-28cfc3e2868980a79d93d2ebdc8747dcb9231f3dd7f2caef96e74107d1ea3bf3 |
| ncoghlan | https://github.com/ncoghlan |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139046649 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139048413 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139048682 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ncoghlan | https://github.com/ncoghlan |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139059262 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139250516 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| https://github.com/ncoghlan |
| ncoghlan | https://github.com/ncoghlan |
|
Sep 15, 2017
| https://github.com/python/cpython/pull/3593#pullrequestreview-62927452 |
|
View reviewed changes
| https://github.com/python/cpython/pull/3593/files/d4d9219d59069af71b6d01d7d8ea32443cc922d8 |
| Python/import.c | https://github.com/python/cpython/pull/3593/files/d4d9219d59069af71b6d01d7d8ea32443cc922d8#diff-28cfc3e2868980a79d93d2ebdc8747dcb9231f3dd7f2caef96e74107d1ea3bf3 |
| ncoghlan | https://github.com/ncoghlan |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139046735 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139048440 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
|
| https://github.com/ericsnowcurrently |
| Return a borrowed reference from _PyImport_GetModuleWithError(). | https://github.com/python/cpython/pull/3593/commits/9c3929574c70f349c0d1029700a8e704064bd762 |
| 9c39295 | https://github.com/python/cpython/pull/3593/commits/9c3929574c70f349c0d1029700a8e704064bd762 |
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#issuecomment-329655252 |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| https://github.com/ncoghlan |
| ncoghlan | https://github.com/ncoghlan |
|
Sep 15, 2017
| https://github.com/python/cpython/pull/3593#pullrequestreview-62940774 |
|
View reviewed changes
| https://github.com/python/cpython/pull/3593/files/9c3929574c70f349c0d1029700a8e704064bd762 |
| ncoghlan | https://github.com/ncoghlan |
| https://github.com/python/cpython/pull/3593#pullrequestreview-62940774 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| Modules/_pickle.c | https://github.com/python/cpython/pull/3593/files/9c3929574c70f349c0d1029700a8e704064bd762#diff-6a8bf1993db0eae81c2ee98e38eeb1ac4d9342b8b5c090f33417a064e6678562 |
| ncoghlan | https://github.com/ncoghlan |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139059331 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139222974 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| Python/import.c | https://github.com/python/cpython/pull/3593/files/9c3929574c70f349c0d1029700a8e704064bd762#diff-28cfc3e2868980a79d93d2ebdc8747dcb9231f3dd7f2caef96e74107d1ea3bf3 |
| ncoghlan | https://github.com/ncoghlan |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#discussion_r139059398 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| September 15, 2017 19:27 | https://github.com/python/cpython/pull/3593#commits-pushed-446acd5 |
|
| https://github.com/ericsnowcurrently |
| Fix some ref leaks. | https://github.com/python/cpython/pull/3593/commits/446acd530af879b37a5cfc9fa2a98caca43ab3f7 |
| 446acd5 | https://github.com/python/cpython/pull/3593/commits/446acd530af879b37a5cfc9fa2a98caca43ab3f7 |
|
| https://github.com/ericsnowcurrently |
| Add a fast path for pickle. | https://github.com/python/cpython/pull/3593/commits/540b99a38d300443d067b1e18130f386a742478a |
| 540b99a | https://github.com/python/cpython/pull/3593/commits/540b99a38d300443d067b1e18130f386a742478a |
|
| https://github.com/ericsnowcurrently |
| Ignore errors from PyObject_GetItem(). | https://github.com/python/cpython/pull/3593/commits/5f0735b431d4d06359793c782bbd6237f35f22f7 |
| 5f0735b | https://github.com/python/cpython/pull/3593/commits/5f0735b431d4d06359793c782bbd6237f35f22f7 |
|
| https://github.com/ericsnowcurrently |
| Factor out _checkmodule. | https://github.com/python/cpython/pull/3593/commits/63beead2f37d4d9abdc1c20fcc7186d149cd3515 |
| 63beead | https://github.com/python/cpython/pull/3593/commits/63beead2f37d4d9abdc1c20fcc7186d149cd3515 |
|
| https://github.com/ericsnowcurrently |
| Drop an extra INCREF. | https://github.com/python/cpython/pull/3593/commits/00f3f24af469ceba80683c781f3e85fc46247f8e |
| 00f3f24 | https://github.com/python/cpython/pull/3593/commits/00f3f24af469ceba80683c781f3e85fc46247f8e |
|
| https://github.com/ericsnowcurrently |
| Drop _PyImport_GetModule() and _PyImport_GetModuleWithError(). | https://github.com/python/cpython/pull/3593/commits/5a7bba299e18b32441a004081ba3f43a3d6c2260 |
| 5a7bba2 | https://github.com/python/cpython/pull/3593/commits/5a7bba299e18b32441a004081ba3f43a3d6c2260 |
|
| https://github.com/ericsnowcurrently |
| tabs to spaces | https://github.com/python/cpython/pull/3593/commits/3a73b521187204c37b9d2d863d141b3bbb4811f2 |
| 3a73b52 | https://github.com/python/cpython/pull/3593/commits/3a73b521187204c37b9d2d863d141b3bbb4811f2 |
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| 3f9eee6 | https://github.com/python/cpython/commit/3f9eee6eb4b25fe1926eaa5f00e02344b126f54d |
| Sep 15, 2017 | https://github.com/python/cpython/pull/3593#event-1251920427 |
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
| September 15, 2017 22:35 | https://github.com/python/cpython/pull/3593#event-1251920589 |
| https://github.com/Mariatta |
| Mariatta | https://github.com/Mariatta |
|
awaiting merge
| https://github.com/python/cpython/issues?q=state%3Aopen%20label%3A%22awaiting%20merge%22 |
| Oct 8, 2017 | https://github.com/python/cpython/pull/3593#event-1283082715 |
| https://github.com/jeremycline |
| jeremycline | https://github.com/jeremycline |
|
Jun 29, 2018
| https://github.com/python/cpython/pull/3593#ref-pullrequest-336703227 |
|
bpo-28411: Fix redundant declaration of _PyImport_AddModuleObject
#7992
| https://github.com/python/cpython/pull/7992 |
| https://github.com/pganssle |
| pganssle | https://github.com/pganssle |
|
Feb 5, 2019
| https://github.com/python/cpython/pull/3593#ref-pullrequest-356555255 |
|
bpo-34572: change _pickle unpickling to use import rather than retrieving from sys.modules
#9047
| https://github.com/python/cpython/pull/9047 |
| https://github.com/ericsnowcurrently |
| ericsnowcurrently | https://github.com/ericsnowcurrently |
|
Apr 10, 2022
| https://github.com/python/cpython/pull/3593#ref-issue-1198960780 |
|
[subinterpreters] Eliminate PyInterpreterState.modules.
#72597
| https://github.com/python/cpython/issues/72597 |
| https://github.com/colesbury |
| colesbury | https://github.com/colesbury |
|
Nov 21, 2025
| https://github.com/python/cpython/pull/3593#ref-pullrequest-3652151623 |
|
gh-137422: Fix race condition in PyImport_AddModuleRef
#141822
| https://github.com/python/cpython/pull/141822 |
| https://github.co/hiddenchars |
| https://github.com/python/cpython/pull/{{ revealButtonHref }} |
| Sign up for free | https://github.com/join?source=comment-repo |
| Sign in to comment | https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F3593 |
|
| https://github.com/ncoghlan |
|
ncoghlan
| https://github.com/ncoghlan |
|
| https://github.com/python/cpython/pull/3593/files/9c3929574c70f349c0d1029700a8e704064bd762 |
| Please reload this page | https://github.com/python/cpython/pull/3593 |
|
| https://github.com/ericsnowcurrently |
|
| https://github.com/ncoghlan |
|
| https://github.com/Mariatta |
|
| https://github.com/the-knights-who-say-ni |
|
| https://github.com/bedevere-bot |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |