[PATCH] sysdeps-unix: use MSG_NOSIGNAL when sending creds

Will Thompson will.thompson at collabora.co.uk
Fri Dec 24 03:01:42 PST 2010


On 24/12/10 10:04, Lennart Poettering wrote:
> If find this rather unnecessary and just confusing. This complicates the
> code for no benefit: MSG_NOSIGNAL is a Linuxism, and on Linux it is a
> macro. Hence checking for it via #ifdef directly is a good thing, and
> simple and safe.
>
> Plead don't unnecessarily clutter the code with autocnf'ism where they
> aren't really necessary!

Ah, I committed the patch to use HAVE_DECL_MSG_NOSIGNAL. I don't think 
it *complicates* the code as such.

I only thought of doing this at all because, while looking at the 
MSG_NOSIGNAL-related patches in the history, I noticed that the commit 
which set _dbus_modify_sigpipe to FALSE conditionally on the presence of 
MSG_NOSIGNAL had already added the autoconf check and used it in 
dbus-connection.c: 
<http://cgit.freedesktop.org/dbus/dbus/commit/?id=03d50fb>. :)

Regards,
-- 
Will


More information about the dbus mailing list