FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-127906: Declare timeval struct in pytime.h on Windows by vstinner · Pull Request #127908 · python/cpython · GitHub

Repository navigation

[3.12] gh-127906: Declare timeval struct in pytime.h on Windows - #127908

Merged
vstinner merged 1 commit into
python:3.12from
vstinner:pytime12
Dec 13, 2024
Merged

vstinner merged 1 commit into
python:3.12from
vstinner:pytime12

Conversation

vstinner commented Dec 13, 2024 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Fix the following MSVC compiler warning:

include\cpython\pytime.h(192): warning C4115: 'timeval':
named type definition in parentheses

Fix the following MSVC compiler warning:

    include\cpython\pytime.h(192): warning C4115: 'timeval':
    named type definition in parentheses

Copy link
Copy Markdown
Member Author

Note: the existing struct timeval code was added for clang by commit 737d367:

+#ifdef __clang__
+struct timeval;
+#endif
commit 737d367b1f4bad76914173a64d6bbe19a984cd5f
Author: Steve Dower <steve.dower@python.org>
Date:   Fri Jan 27 14:45:08 2023 +0000

    gh-77532: Minor tweaks to allow compiling with PlatformToolset=ClangCL on Windows (GH-101352)
    
    To use this, ensure that clang support was selected in Visual Studio Installer, then set the PlatformToolset environment variable to "ClangCL" and build as normal from the command line.
    It remains unsupported, but at least is possible now for experimentation.

vstinner enabled auto-merge (squash) December 13, 2024 12:07
vstinner merged commit c77bfd7 into python:3.12 Dec 13, 2024
vstinner deleted the pytime12 branch December 13, 2024 12:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL