[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH gnulib 2/4] accept4, fcntl, w32sock: Avoid warnings on x86_64
From: |
Bruno Haible |
Subject: |
Re: [PATCH gnulib 2/4] accept4, fcntl, w32sock: Avoid warnings on x86_64 mingw64. |
Date: |
Sat, 28 Jan 2012 13:27:31 +0100 |
User-agent: |
KMail/4.7.4 (Linux/3.1.0-1.2-desktop; KDE/4.7.4; x86_64; ; ) |
Marc-André Lureau wrote:
> 2012-01-24 Marc-André Lureau <address@hidden>
>
> + accept4, fcntl, w32sock: Avoid warnings on x86_64 mingw64.
> + * lib/accept4.c (accept4): _open_osfhandle() expect a
> + intptr_t.
> + * lib/fcntl.c (dupfd): Likewise.
> + * lib/w32sock.h (SOCKET_TO_FD): Likewise.
Thanks. I have verified that the argument type of _open_osfhandle() is
'long' only in the primary mingw and is 'intptr_t' in MSVC9 and mingw64,
and that your change does not need a #include <stdint.h> (since the
use of intptr_t is limited to native Windows platforms).
Applied with a slightly modified ChangeLog entry. A ChangeLog entry should
mention the change that was done; just mentioning a fact or rationale without
mentioning the change can be confusing.
Bruno
2012-01-28 Marc-André Lureau <address@hidden> (tiny change)
accept4, fcntl, socket modules: Avoid warnings on x86_64 mingw64.
* lib/accept4.c (accept4): Use intptr_t to convert handle pointer to
an integer.
* lib/fcntl.c (dupfd): Likewise.
* lib/w32sock.h (SOCKET_TO_FD): Likewise.
- [PATCH gnulib 0/4] some mingw64 warnings fixes, Marc-André Lureau, 2012/01/25
- [PATCH gnulib 1/4] malloca: Avoid warnings on x86_64 mingw64., Marc-André Lureau, 2012/01/25
- [PATCH gnulib 3/4] select, poll, isatty: Avoid warnings on x86_64 mingw64., Marc-André Lureau, 2012/01/25
- [PATCH gnulib 2/4] accept4, fcntl, w32sock: Avoid warnings on x86_64 mingw64., Marc-André Lureau, 2012/01/25
- Re: [PATCH gnulib 2/4] accept4, fcntl, w32sock: Avoid warnings on x86_64 mingw64.,
Bruno Haible <=
- [PATCH gnulib 4/4] RFC stat: x86_64 mingw64 #define stat as _stat64., Marc-André Lureau, 2012/01/25