Wireshark mailing list archives
io.h include guards
From: Graham Bloice <graham.bloice () trihedral com>
Date: Thu, 5 Nov 2015 21:44:55 +0000
README.developer states
Don't include <unistd.h> without protecting it with
#ifdef HAVE_UNISTD_H
...
#endif
and, if you're including it to get routines such as "open()", "close()",
"read()", and "write()" declared, also include <io.h> if present:
#ifdef HAVE_IO_H
#include <io.h>
#endif
so looking at io.h, it's included by 5 files, only one of which
(ascend_scanner.c) adds the HAVE_IO_H guard, all the others use a _WIN32
guard.
ConfigureChecks.cmake doesn't run a check for io.h, and cmakeconfig.h.in
doesn't have a #cmakdefine for HAVE_IO_H.
What's the best way to make this all consistent, fix CMake and the _WIN32
guards to use HAVE_IO_H, or change the docs and ascend_scanner to use
_WIN32?
--
Graham Bloice
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- io.h include guards Graham Bloice (Nov 05)
- Re: io.h include guards Guy Harris (Nov 05)
