-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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 warnings in static inline functions in installed headers #8870
fix warnings in static inline functions in installed headers #8870
Conversation
= { 0 } has different meaning in C and C++: in C it initialises all members to 0, in C++ only the first one so the compiler warns that other members are not initialised. Use an empty initialiser list instead.
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for that, I've been bothered by it too.
@freerdp-bot can you test that my friend ? |
Refer to this link for build results (access rights to CI server needed): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already changed in #8866
*pcbe = empty; | ||
pcbe->Version = 1; | ||
} | ||
|
||
static INLINE VOID DestroyThreadpoolEnvironment(PTP_CALLBACK_ENVIRON pcbe) | ||
{ | ||
/* no actions, this may change in a future release. */ | ||
(void)pcbe; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better use WINPR_UNUSED
Looking at these functions, it might be the better option to make them normal functions. |
= { 0 } has different meaning in C and C++: in C it initialises all members to 0, in C++ only the first one so the compiler warns that other members are not initialised. Use an empty initialiser list instead.
It's not very pretty; if you prefer a different style let me know :)