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: always ensure K&R functions have at least 1 declaration #246

Merged
merged 1 commit into from
Dec 28, 2024

Conversation

amaanq
Copy link
Member

@amaanq amaanq commented Dec 28, 2024

Problem

Functions with empty parameters are getting parsed internally as _old_style_function_definition, causing seemingly-correct queries to fail.

Solution

For "old-style" K&R functions, there should be at least 1 declaration following the declarator, no declarators should parse as a normal function_definition

@amaanq amaanq merged commit 3aa2995 into master Dec 28, 2024
4 checks passed
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.

Query with supertype fails when it shouldn't
1 participant