[libcxx] r194870 - Windows.h is not required.

Marshall Clow mclow.lists at gmail.com
Fri Nov 15 15:02:54 PST 2013


On Nov 15, 2013, at 2:54 PM, Yaron Keren <yaron.keren at gmail.com> wrote:

> Author: yrnkrn
> Date: Fri Nov 15 16:54:15 2013
> New Revision: 194870
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=194870&view=rev
> Log:
> Windows.h is not required.
> 
> Modified:
>    libcxx/trunk/include/support/win32/support.h
> 
> Modified: libcxx/trunk/include/support/win32/support.h
> URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/support/win32/support.h?rev=194870&r1=194869&r2=194870&view=diff
> ==============================================================================
> --- libcxx/trunk/include/support/win32/support.h (original)
> +++ libcxx/trunk/include/support/win32/support.h Fri Nov 15 16:54:15 2013
> @@ -23,7 +23,6 @@
> #ifndef NOMINMAX
> #define NOMINMAX
> #endif
> -#include <Windows.h>
> 
> extern "C" {

If <windows.h> is not required, is the whole "#ifndef NOMINMAX" stuff not required either?


-- Marshall

Marshall Clow     Idio Software   <mailto:mclow.lists at gmail.com>

A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait).
        -- Yu Suzuki





More information about the cfe-commits mailing list