Title: CPython3.12a1 still has special case support for `PyStringMap` · Issue #217 · jython/jython · GitHub
Open Graph Title: CPython3.12a1 still has special case support for `PyStringMap` · Issue #217 · jython/jython
X Title: CPython3.12a1 still has special case support for `PyStringMap` · Issue #217 · jython/jython
Description: Hi! I am CPython contributor :) And I've recently noticed that for the last 22 years we have some special casing for org.python.core.PyStringMap. Here are they: https://github.com/python/cpython/blame/db115682bd639a2642c617f0b7d5b30cd7d7...
Open Graph Description: Hi! I am CPython contributor :) And I've recently noticed that for the last 22 years we have some special casing for org.python.core.PyStringMap. Here are they: https://github.com/python/cpython/bl...
X Description: Hi! I am CPython contributor :) And I've recently noticed that for the last 22 years we have some special casing for org.python.core.PyStringMap. Here are they: https://github.com/python/cpytho...
Opengraph URL: https://github.com/jython/jython/issues/217
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"CPython3.12a1 still has special case support for `PyStringMap`","articleBody":"Hi! I am CPython contributor :)\r\nAnd I've recently noticed that for the last 22 years we have some special casing for `org.python.core.PyStringMap`.\r\n\r\nHere are they:\r\n- https://github.com/python/cpython/blame/db115682bd639a2642c617f0b7d5b30cd7d7f472/Lib/copy.py#L59-L62\r\n- https://github.com/python/cpython/blame/db115682bd639a2642c617f0b7d5b30cd7d7f472/Lib/pickle.py#L101-L105\r\n- https://github.com/python/cpython/blame/db115682bd639a2642c617f0b7d5b30cd7d7f472/Lib/xml/sax/__init__.py#L98-L100\r\n\r\nThe question is: do we still need it?\r\n\r\nSome concerns:\r\n1. It imports code from some third-party module in the stdlib. This is the only place (there might be some other cases that I am not familiar with, please proove me wrong) where we do it. It might be a source of some unexpected results if user has the same package structure. It might be even related to security, because importing third-party modules is never safe in Python\r\n2. Jython seems to only support 2.7 a the moment, so we might not need this anymore in our core library 🤔 \r\n\r\nPlease, share your vision on this 🙂 ","author":{"url":"https://github.com/sobolevn","@type":"Person","name":"sobolevn"},"datePublished":"2022-11-14T09:02:11.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/217/jython/issues/217"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:e5d9f6e7-afec-a9ee-c2e6-0e4687e0a442 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | D630:19DA02:232B584:31266B0:696940B1 |
| html-safe-nonce | 8b7af62328c47259c8823e8f7ccc3dc4fb9462fa9e8aeb21e59fb9823f2ecde9 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJENjMwOjE5REEwMjoyMzJCNTg0OjMxMjY2QjA6Njk2OTQwQjEiLCJ2aXNpdG9yX2lkIjoiMjQ5NjA4NTk4NDYwODMzODA5NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 4378694a44d331c195fbfd40630b6a205822cd881d58248af2d150b4dac3295d |
| hovercard-subject-tag | issue:1447655887 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/jython/jython/217/issue_layout |
| twitter:image | https://opengraph.githubassets.com/1b282bdaccc33d3f93f18f656ce3b167363d846ebbe81255f2b4ed1c26786249/jython/jython/issues/217 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/1b282bdaccc33d3f93f18f656ce3b167363d846ebbe81255f2b4ed1c26786249/jython/jython/issues/217 |
| og:image:alt | Hi! I am CPython contributor :) And I've recently noticed that for the last 22 years we have some special casing for org.python.core.PyStringMap. Here are they: https://github.com/python/cpython/bl... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | sobolevn |
| hostname | github.com |
| expected-hostname | github.com |
| None | 54182691a21263b584d2e600b758e081b0ff1d10ffc0d2eefa51cf754b43b51d |
| turbo-cache-control | no-preview |
| go-import | github.com/jython/jython git https://github.com/jython/jython.git |
| octolytics-dimension-user_id | 10533356 |
| octolytics-dimension-user_login | jython |
| octolytics-dimension-repository_id | 282480317 |
| octolytics-dimension-repository_nwo | jython/jython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 282480317 |
| octolytics-dimension-repository_network_root_nwo | jython/jython |
| 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 | d69ac0477df0f87da03b8b06cebd187012d7a930 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width