Title: gh-99191: Use correct check for MSVC C++ version support in _wmimodule.cpp by CAM-Gerlach · Pull Request #100381 · python/cpython · GitHub
Open Graph Title: gh-99191: Use correct check for MSVC C++ version support in _wmimodule.cpp by CAM-Gerlach · Pull Request #100381 · python/cpython
X Title: gh-99191: Use correct check for MSVC C++ version support in _wmimodule.cpp by CAM-Gerlach · Pull Request #100381 · python/cpython
Description: As discussed in #99191 , in PC/_wmimodule.cpp it currently uses the check #if _MSC_VER >= 1929 to determine if the compiler supports C++20 designated initializers. However, the /std:c++20 flag was only introduced in VS 2019 16.11, but 1929 is also the version reported in VS 2019 16.10, which doesn't support that flag. Therefore, the current check will break (resulting in the unsupported C++20 branch being taken and presumably resulting in a compiler error) on VS 16.10 and also if the build is invoked such that \std:c++20 is overridden or not passed. However, we can instead use _MSVC_LANG >= 202002L to check whether C++20 or greater is actually being targeted. Fixes #99191 Issue: gh-99191
Open Graph Description: As discussed in #99191 , in PC/_wmimodule.cpp it currently uses the check #if _MSC_VER >= 1929 to determine if the compiler supports C++20 designated initializers. However, the /std:c++20 flag w...
X Description: As discussed in #99191 , in PC/_wmimodule.cpp it currently uses the check #if _MSC_VER >= 1929 to determine if the compiler supports C++20 designated initializers. However, the /std:c++20 fl...
Opengraph URL: https://github.com/python/cpython/pull/100381
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:acb01644-bc20-e719-9fa3-cf2bce0d6ca0 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | DC50:333370:1CE7799:285D454:696A5B89 |
| html-safe-nonce | febdaf921639a7156a0b9ce17e7e590afc5d1a19543ca3786608eae2df6f0b35 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEQzUwOjMzMzM3MDoxQ0U3Nzk5OjI4NUQ0NTQ6Njk2QTVCODkiLCJ2aXNpdG9yX2lkIjoiMTUyMzc2Njc5Njk3Mzc0MDg5IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | ffe946e6680e1ac7d7dba3fff77866ac9a2c6953cac50d77fba3e724da5772a7 |
| hovercard-subject-tag | pull_request:1172948705 |
| 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/100381/files |
| twitter:image | https://avatars.githubusercontent.com/u/17051931?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/17051931?s=400&v=4 |
| og:image:alt | As discussed in #99191 , in PC/_wmimodule.cpp it currently uses the check #if _MSC_VER >= 1929 to determine if the compiler supports C++20 designated initializers. However, the /std:c++20 flag w... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 3f871c8e07f0ae1886fa8dac284166d28b09ad5bada6476fc10b674e489788ef |
| 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 | 63c426b30d262aba269ef14c40e3c817b384cd61 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width