Title: bpo-39943: Fix MSVC warnings in sre extension by ammaraskar · Pull Request #20508 · python/cpython · GitHub
Open Graph Title: bpo-39943: Fix MSVC warnings in sre extension by ammaraskar · Pull Request #20508 · python/cpython
X Title: bpo-39943: Fix MSVC warnings in sre extension by ammaraskar · Pull Request #20508 · python/cpython
Description: While the PyMem_Del change seems harmless enough, I'm a little bit uneasy about the memcpy one. This file is pretty old and unlikely to change in the future but it seems like it opens up the possibility of someone making a mistake like DATA_STACK_POP(state, 5, ...) and have it not caught by the type checker because of the explicit cast. For that particular one we could guard the explicitly casted version with #ifdef _MSC_VER or something so it's harder to make that mistake and not have it be caught by the CI. https://bugs.python.org/issue39943
Open Graph Description: While the PyMem_Del change seems harmless enough, I'm a little bit uneasy about the memcpy one. This file is pretty old and unlikely to change in the future but it seems like it opens up the po...
X Description: While the PyMem_Del change seems harmless enough, I'm a little bit uneasy about the memcpy one. This file is pretty old and unlikely to change in the future but it seems like it opens up th...
Opengraph URL: https://github.com/python/cpython/pull/20508
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:371d0afa-a824-053c-166c-a1eb3f766b77 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | DF4A:159D04:A815AC:DC5820:696B56B5 |
| html-safe-nonce | c4e1af6b3e8ea813977916fd4de036aa8383194553101236967aa935152107d8 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJERjRBOjE1OUQwNDpBODE1QUM6REM1ODIwOjY5NkI1NkI1IiwidmlzaXRvcl9pZCI6Ijc2OTUyNjQ0MjE3ODg3MzUxNTciLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 921ae2145355dc8fb2cede809f5363232dc6adf46494a5fb06989022abcaf8b1 |
| hovercard-subject-tag | pull_request:424989403 |
| 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 | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/20508/files |
| twitter:image | https://avatars.githubusercontent.com/u/773529?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/773529?s=400&v=4 |
| og:image:alt | While the PyMem_Del change seems harmless enough, I'm a little bit uneasy about the memcpy one. This file is pretty old and unlikely to change in the future but it seems like it opens up the po... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f99f7c1d70f01da5b93e5ca90303359738944d8ab470e396496262c66e60b8d |
| turbo-cache-control | no-preview |
| diff-view | unified |
| 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 full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 82560a55c6b2054555076f46e683151ee28a19bc |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width