Skip to content

gh-148775: Bump c-analyzer MAX_SIZES for Modules/clinic/_testclinic.c.h#148776

Open
ashm-dev wants to merge 1 commit into
python:mainfrom
ashm-dev:fix-c-analyzer-testclinic-maxsize
Open

gh-148775: Bump c-analyzer MAX_SIZES for Modules/clinic/_testclinic.c.h#148776
ashm-dev wants to merge 1 commit into
python:mainfrom
ashm-dev:fix-c-analyzer-testclinic-maxsize

Conversation

@ashm-dev
Copy link
Copy Markdown
Contributor

@ashm-dev ashm-dev commented Apr 19, 2026

make check-c-globals fails on main because Modules/clinic/_testclinic.c.h (auto-generated by Argument Clinic) has grown past the MAX_SIZES limit set in Tools/c-analyzer/cpython/_parser.py. Bump 120_000200_000.

No Misc/NEWS.d entry — tooling-only change to the c-analyzer parser limits. Please apply the skip news label.

Fixes #148775.

@ashm-dev
Copy link
Copy Markdown
Contributor Author

@ericsnowcurrently could you please review this PR?

@ashm-dev ashm-dev force-pushed the fix-c-analyzer-testclinic-maxsize branch from f0366cc to 0094b25 Compare May 19, 2026 11:25
@ashm-dev
Copy link
Copy Markdown
Contributor Author

Friendly ping — rebased onto current main (the previous 125_000 bump turned out to be insufficient: Modules/clinic/_testclinic.c.h is now ~137k bytes, so 200_000 leaves real headroom). One-line diff, no conflicts. @ericsnowcurrently @vstinner could one of you take a quick look?

@ashm-dev
Copy link
Copy Markdown
Contributor Author

The Ubuntu (free-threading) / build and test failure is unrelated — the runner received a shutdown signal mid-run (exit code 143, "runner has received a shutdown signal"). All tests passed up to that point. A rerun should be green.

@vstinner
Copy link
Copy Markdown
Member

I fail to reproduce the issue: #148775 (comment).

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.

c-analyzer: bump MAX_SIZES for Modules/clinic/_testclinic.c.h

3 participants