Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-133228: c-analyzer clang preprocessor#133229

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
dg-pb wants to merge5 commits intopython:main
base:main
Choose a base branch
Loading
fromdg-pb:c_analyzer_clang_experimental_preprocessor

Conversation

dg-pb
Copy link
Contributor

@dg-pbdg-pb commentedApr 30, 2025•
edited
Loading

Few minor deviations from gcc, but all seems to be working correctly.

Changes (apart from main implementation):

  1. MAX_SIZES additions for 2 files were required.
  2. Added a fix for [n]curses.h failures on clang (gcc also includes both markerlines - curses.h and ncurses.h, but a bit differently, thus passes)
  3. Added couple of global ignores that were failing on clang (don't know why they weren't on gcc)

@dg-pb
Copy link
ContributorAuthor

dg-pb commentedMay 1, 2025•
edited
Loading

In addition getting the following 2 errors:

Modules/socketmodule.c - netdb_lock not supported (typespec) static PyThread_type_lockPython/bootstrap_hash.c py_getentropy getentropy_works not supported (mutable) static int

@dg-pb
Copy link
ContributorAuthor

In addition getting the following 2 errors:

Modules/socketmodule.c - netdb_lock not supported (typespec) static PyThread_type_lockPython/bootstrap_hash.c py_getentropy getentropy_works not supported (mutable) static int

Included these 2 into tsvs. Not sure why they were not failing in tests. Can only run clang, not gcc, so can not tell.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@AA-TurnerAA-TurnerAA-Turner left review comments

@ericsnowcurrentlyericsnowcurrentlyAwaiting requested review from ericsnowcurrentlyericsnowcurrently is a code owner

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@dg-pb@AA-Turner
[8]ページ先頭
©2009-2025 Movatter.jp