Title: bpo-9566: Change HANDLE argument parsing to unsigned in msvcrtmodule.c by segevfiner · Pull Request #2904 · python/cpython · GitHub
Open Graph Title: bpo-9566: Change HANDLE argument parsing to unsigned in msvcrtmodule.c by segevfiner · Pull Request #2904 · python/cpython
X Title: bpo-9566: Change HANDLE argument parsing to unsigned in msvcrtmodule.c by segevfiner · Pull Request #2904 · python/cpython
Description: A fix after #2492, I guess the values were truncated before. Tests are failing in Windows x64. I ran the test suite locally (Default flags) making sure it passes now with the correct Python version. Barring weird failure from test.test_uuid.TestInternals.test_ipconfig_getnode related to encoding, I think subprocess uses cp1255 while the console seems to be set to cp862 in my computer. Maybe Python should be deciding on the code page for use by subprocess by using GetConsoleOutputCP. Who knows if that's always the right thing to do... 😓 @zooba @Haypo https://bugs.python.org/issue9566
Open Graph Description: A fix after #2492, I guess the values were truncated before. Tests are failing in Windows x64. I ran the test suite locally (Default flags) making sure it passes now with the correct Python version...
X Description: A fix after #2492, I guess the values were truncated before. Tests are failing in Windows x64. I ran the test suite locally (Default flags) making sure it passes now with the correct Python version...
Opengraph URL: https://github.com/python/cpython/pull/2904
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:64ad6960-dccf-ae54-05f8-f9a5edd280bc |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | C7D2:20ED06:DAC9B5:123480F:6A52AA80 |
| html-safe-nonce | d03cdcc15e83eefa25159562985a9843b3e868d7cbe3d1a5879f1adf75fcb9d8 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDN0QyOjIwRUQwNjpEQUM5QjU6MTIzNDgwRjo2QTUyQUE4MCIsInZpc2l0b3JfaWQiOiI1ODI3NTE4NzU0MDcwODk5MzI5IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 969bbecf9126633a191012ddbc2a748ce6bc8a92920c048cdc21b4f94207b71c |
| hovercard-subject-tag | pull_request:132634995 |
| 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/2904/files |
| twitter:image | https://avatars.githubusercontent.com/u/24731903?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/24731903?s=400&v=4 |
| og:image:alt | A fix after #2492, I guess the values were truncated before. Tests are failing in Windows x64. I ran the test suite locally (Default flags) making sure it passes now with the correct Python version... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb |
| 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 | 07a982c1d40157c619b364352b704c3ce66bb332 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width