[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add missing include of windows.h in nl_langinfo.c
|
From: |
Eric Blake |
|
Subject: |
Re: [PATCH] Add missing include of windows.h in nl_langinfo.c |
|
Date: |
Thu, 10 Jul 2014 09:51:10 -0600 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 |
On 07/10/2014 09:21 AM, Eli Zaretskii wrote:
>> Date: Wed, 09 Jul 2014 16:02:31 -0600
>> From: Eric Blake <address@hidden>
>>
>>> Signed-off-by: Pavel Hrdina <address@hidden>
>>> ---
>>> lib/nl_langinfo.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>
>> ACK, fix pushed.
>
> I believe this commit caused redundant inclusion of header files. I
> suggest the simple follow-up patch below.
>
> nl_langinfo: Fix last change.
>
> Don't include stdio.h and windows.h twice on MS-Windows. Also, define
> WIN32_LEAN_AND_MEAN before including windows.h.
Thanks; I've pushed this followup.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature