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

fixes with clang compiler #928

Closed
wants to merge 1 commit into from
Closed

fixes with clang compiler #928

wants to merge 1 commit into from

Conversation

kraj
Copy link
Contributor

@kraj kraj commented Aug 30, 2022

No description provided.

clang-15+ has started diagnosing them as errors

thread.c:925:18: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
| void STATS_UNLOCK() {
|                  ^
|                   void

Signed-off-by: Khem Raj <raj.khem@gmail.com>
@dormando
Copy link
Member

Thanks!

@debohman
Copy link
Contributor

Which version of clang are you using?

@kraj
Copy link
Contributor Author

kraj commented Oct 13, 2022

clang-15, but it was before the error were turned back into warnings since it was delayed till clang-16.

@debohman
Copy link
Contributor

So, are you building with a final release of clang-15? Because, memcached-1.6.17 does not build for me using clang-15.0.2. The build has:

clang -Wall -Werror -pedantic -Wmissing-prototypes -Wmissing-declarations -Wredundant-decls

@kraj
Copy link
Contributor Author

kraj commented Oct 13, 2022

So, are you building with a final release of clang-15? Because, memcached-1.6.17 does not build for me using clang-15.0.2. The build has:

clang -Wall -Werror -pedantic -Wmissing-prototypes -Wmissing-declarations -Wredundant-decls

yes I am on latest commit on release/15.x branch

@dormando
Copy link
Member

released in 1.6.18

@dormando dormando closed this Jan 11, 2023
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.

None yet

3 participants