[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Chicken-hackers] [PATCH 2/2] Invert poll(2) flag default

From: Peter Bex
Subject: Re: [Chicken-hackers] [PATCH 2/2] Invert poll(2) flag default
Date: Wed, 6 Aug 2014 08:24:25 +0200
User-agent: Mutt/

On Mon, Aug 04, 2014 at 06:10:46PM +0200, Moritz Heidkamp wrote:
> The attached patch is a follow-up to my previous patch (the one which
> enables poll(2) on Android). It inverts the HAVE_POSIX_POLL flag to
> HAVE_NO_POSIX_POLL which only needs to be set when poll(2) is *not*
> available rather than the other way around. The purpose is to make the
> safe choice the default. 

This seems like a good idea.  However, could you also swap the two
code blocks?  A double negation (#ifndef NO_...) can be confusing, and
by making it read "#ifdef NO_POSIX_POLL" (I'd probably drop the
HAVE_ prefix, as that's more idiomatic AFAICT), it becomes a little


reply via email to

[Prev in Thread] Current Thread [Next in Thread]