<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Sep 11, 2014 at 10:32 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 11 September 2014 13:29, Eran Ifrah <<a href="mailto:eran.ifrah@gmail.com">eran.ifrah@gmail.com</a>> wrote:<br>
> The header file is included indirectly (<winnt.h>). However, the definitions<br>
> are not visible since _WIN32_WINNT is set to a value lower than 0x0502<br>
> Attached is a patch to the CMakeLists.txt file that adds<br>
> -D_WIN32_WINNT=0x0502 in case of MinGW and WIN32 conditions are met<br>
<br>
</span>Not sure which one is better. Aaron, any thoughts?</blockquote><div><br></div><div>Given that there are approximately *two* places where we include windows.h / winnt.h, I think we should define the macro in the .cpp file rather than adding a flag for every TU. It appears we don't define WINVER & co anywhere else, so let's not change that practice.</div><div><br></div><div>I also don't think this needs to be MinGW specific. It's just that the Win7 / Win8 SDKs we use don't support pre-XP OSs with the #ifndef directives that MinGW appears to use.</div></div></div></div>