[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] Allow AF_UNIX sockets to be disabled on non
Re: [Qemu-devel] Re: [PATCH] Allow AF_UNIX sockets to be disabled on non-Windows
Wed, 6 Feb 2008 22:47:50 +0000
Anthony Liguori, le Wed 06 Feb 2008 13:19:23 -0600, a écrit :
> Ian Jackson wrote:
> >iwj writes ("[PATCH] Allow AF_UNIX sockets to be disabled on non-Windows"):
> >>The patch below makes it possible to disable AF_UNIX (unix-domain)
> >>sockets in host environments which do not define _WIN32, by adding
> >>-DNO_UNIX_SOCKETS to the compiler flags. This is useful in the
> >>effectively-embedded qemu host which are going to be using for device
> >>emulation in Xen.
> It should just check a define for _MINIOS.
That's exactly what we wanted to avoid.
> That makes it a lot more obvious why it's not being included.
But it doesn't necessarily make obvious _what_ is not being included
(here, local sockets). To my mind, something like
#if !(defined(_WIN32) || defined(_MINIOS))
And then in the code, #ifdef DO_UNIX_SOCKET, is much nicer than
repeating the if (!def||def) everywhere (and have to change them all if
another system needs that too)