Skip to content

Add missing (void) on functions without args#442

Merged
athre0z merged 1 commit intomasterfrom
add-missing-void
May 25, 2023
Merged

Add missing (void) on functions without args#442
athre0z merged 1 commit intomasterfrom
add-missing-void

Conversation

@athre0z
Copy link
Copy Markdown
Member

@athre0z athre0z commented May 21, 2023

Multiple functions without arguments were missing the (void) that prevents them from being interpreted as not having a prototype. Newer Clang versions started printing warnings for that in pedantic mode.

Corresponding Zycore PR: https://github.com/zyantific/zycore-c/pull/64/files

Multiple functions without arguments were missing the `(void)` that
prevents them from being interpreted as not having a prototype. Newer
Clang versions started printing warnings for that in pedantic mode.
@athre0z athre0z added the C-bug Category: This is a bug (or a fix for a bug, when applied to PRs) label May 21, 2023
@athre0z athre0z requested a review from flobernd May 21, 2023 19:39
Copy link
Copy Markdown
Member

@flobernd flobernd left a comment

Choose a reason for hiding this comment

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

LGTM!

@athre0z athre0z merged commit d4c37ae into master May 25, 2023
@athre0z athre0z deleted the add-missing-void branch May 25, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-bug Category: This is a bug (or a fix for a bug, when applied to PRs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants