Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix TypeDeclExt/ArrayDeclExt #77

Merged
merged 3 commits into from
Sep 11, 2023
Merged

Conversation

douglas-raillard-arm
Copy link
Contributor

No description provided.

pycparser's TypeDecl node gained an "align" __init__ parameter, so
forward it and use named parameters to avoid silent breakage in the
future.

The issue manifested as a Coord object being used for "type".
Use named parameters to avoid issues like TypDecl silently growing a new
parameter, and pycparserext's parameter list being shifted.
@inducer inducer enabled auto-merge (rebase) September 8, 2023 18:10
@inducer
Copy link
Owner

inducer commented Sep 8, 2023

Thx!

@inducer inducer disabled auto-merge September 8, 2023 18:11
@inducer
Copy link
Owner

inducer commented Sep 8, 2023

Could you look into the test failures?

@douglas-raillard-arm
Copy link
Contributor Author

Looks like the align parameter was added to TypeDecl in 2.21: eliben/pycparser@14ffe7b

I updated the PR with a pycparser==2.21 version bound

@douglas-raillard-arm
Copy link
Contributor Author

@inducer PR updated with ~= instead of == in setup.py

@inducer inducer enabled auto-merge (rebase) September 11, 2023 13:41
@inducer
Copy link
Owner

inducer commented Sep 11, 2023

Thx!

@inducer inducer merged commit 1c5ca53 into inducer:main Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants