Re: COMCTL32: Fix some gcc 4.1 warnings caused by windowsx.h macros.

[Rob Shearman]

Mike McCormack wrote:
> ChangeLog:
> Fix some gcc 4.1 warnings caused by windowsx.h macros.

I see little benefit in this warning being enabled.

IMHO, it would be better to see one patch that turns this warning off 
than ten patches that obfuscate the code

Rob

COMCTL32: Fix some gcc 4.1 warnings caused by windowsx.h macros.

[Mike McCormack]

Rob Shearman wrote:
> I see little benefit in this warning being enabled.
>
> IMHO, it would be better to see one patch that turns this warning off 
> than ten patches that obfuscate the code

Yesterday, Alexandre fixed 4 problems that showed using this warning:

http://source.winehq.org/git/?p=wine.git;a=commit;h=c9cc7e33ffaaf9b359d6bccf...

They showed up with gcc 4.1, but I didn't notice them because of the 
warnings from function casts.  It's probably possible to turn it off 
using -Wno-unused-value, but there are real problems that will be caught 
with it.  Consider part of Alexandre's patch yesterday:

+++ b/dlls/ole32/ole2.c
@@ -801,7 +801,7 @@ static HRESULT WINAPI EnumOLEVERB_Next(
          rgelt->grfAttribs = atolW(pwszAttribs);

          if (pceltFetched)
-            *pceltFetched++;
+            (*pceltFetched)++;
          This->index++;
      }
      return hr;

Getting rid of the relative small number of macro uses that cause the 
warning doesn't seem to difficult, or to obfuscate the code too much to me.

Mike

COMCTL32: Fix some gcc 4.1 warnings caused by windowsx.h macros.

[Thomas Weidenmueller]

Rob Shearman wrote:
> IMHO, it would be better to see one patch that turns this warning off
> than ten patches that obfuscate the code

I absolutely appreciate this warning, IMO it's not code obfuscation.

- Thomas