<div dir="ltr"><div>Hi Yaron</div><div> </div><div>Yes that's true. Windows.h isn't required any more for clang or visual studio builds either.</div><div> </div><div>I didn't remove it in my prior patch to support.h because I was trying to keep my changes minimal for the goal of that patch and remove windows.h in a follow up one as it makes a big difference to compile times, especially Visual Studio and curiously reduces the error count too.</div>
<div> </div><div>I think these lines are also related to windows.h</div><div>//#ifndef NOMINMAX<br>//#define NOMINMAX<br>//#endif</div><div>but min/max might get brought in by other Win32 headers, I'm not sure. It's arguable that this macro should be defined in __config in the Win32 section so it becomes project wide. libcxx goes to some lengths to avoid other min/max issues in (some) other headers in using __undef_min_max.</div>
<div> </div></div>