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

GH-131296: fix clang-cl warning on Windows in semaphore.c by chris-eibl · Pull Request #131595 · python/cpython · GitHub

/ cpython Public

GH-131296: fix clang-cl warning on Windows in semaphore.c - #131595

Merged
gpshead merged 1 commit into
python:mainfrom
chris-eibl:fix_clangcl_semaphore
Mar 22, 2025
ConversationCommits (1)ChecksFiles changed
Merged

GH-131296: fix clang-cl warning on Windows in semaphore.c#131595
gpshead merged 1 commit into
python:mainfrom
chris-eibl:fix_clangcl_semaphore

Conversation

chris-eibl commented Mar 22, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

I think this is a skip news?

Fixes https://github.com/python/cpython/actions/runs/14006628024/job/39221297660#step:4:259

..\Modules\_multiprocessing\semaphore.c(636,36): warning : incompatible pointer types passing 'int *' 
to parameter of type 'long *' [-Wincompatible-pointer-types] [D:\a\cpython\cpython\PCbuild\_multiprocessing.vcxproj]
..\Modules\_multiprocessing\semaphore.c(668,36): warning : incompatible pointer types passing 'int *'
to parameter of type 'long *' [-Wincompatible-pointer-types]

chris-eibl requested a review from gpshead as a code owner March 22, 2025 20:55
bedevere-app Bot mentioned this pull request Mar 22, 2025
Closed
gpshead merged commit bc26f95 into python:main Mar 22, 2025
chris-eibl deleted the fix_clangcl_semaphore branch March 23, 2025 07:04
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

gpshead gpshead approved these changes

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL