Skip to content

appease a few compiler warnings (-Wstrict-prototypes) #4465

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 10, 2025
Merged

appease a few compiler warnings (-Wstrict-prototypes) #4465

merged 1 commit into from
Jul 10, 2025

Conversation

yamt
Copy link
Collaborator

@yamt yamt commented Jul 8, 2025

No description provided.

lum1n0us
lum1n0us approved these changes Jul 9, 2025
View reviewed changes
Copy link
Collaborator

@lum1n0us lum1n0us left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yamt Do you think we should include -Wstrict-prototypes in the global warnings? #4397

@yamt
Copy link
Collaborator Author

yamt commented Jul 9, 2025

@yamt Do you think we should include -Wstrict-prototypes in the global warnings? #4397

maybe.
i noticed this during nuttx build, which has it's own set of compiler options.

@lum1n0us lum1n0us merged commit 933e49d into bytecodealliance:main Jul 10, 2025
451 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lum1n0us lum1n0us lum1n0us approved these changes

@TianlongLiang TianlongLiang TianlongLiang approved these changes

@loganek loganek Awaiting requested review from loganek loganek is a code owner

@no1wudi no1wudi Awaiting requested review from no1wudi no1wudi is a code owner

@wenyongh wenyongh Awaiting requested review from wenyongh wenyongh is a code owner

@xujuntwt95329 xujuntwt95329 Awaiting requested review from xujuntwt95329 xujuntwt95329 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yamt @lum1n0us @TianlongLiang