[PATCH] configure: Check for -Wno-pragma-pack

Martin Storsjö martin at martin.st
Mon Jan 22 15:42:07 CST 2018


On Mon, 22 Jan 2018, Thomas Faber wrote:

> On 2018-01-21 23:05, Martin Storsjo wrote:
>> Recent clang versions warn if an include file changes the pragma
>> pack level without restoring it before the end of the include file.
>> 
>> This avoids warnings for each include of pshpack*.h/poppack.h.
>
> Do you think a #pragma clang diagnostic inside those headers would be 
> feasible, to avoid hiding true positive instances of this warning?

I tested this now. It's easy to get rid of the warnings from the 
pshpack*.h files (by e.g. #pragma clang diagnostic push, #pragma clang 
diagnostic ignored "-Wpragma-pack").

Bbut if I do "#pragma clang diagnostic pop" within poppack.h, the warning 
is enabled again once the end of poppack.h is reached, and the 
warning gets triggered at the end of the source file. So each 
pair of pshpack*.h/poppack.h would need disabling the warning 
ignoring again after the #include "poppack.h".

Do you have any suggestions on how to handle that better, or is it just 
simplest to go for the blanket ignore?

// Martin



More information about the wine-devel mailing list