Greetings.

On Mon, 06 May 2013 19:48:37 +0200 Strake <strake...@gmail.com> wrote:
> On 06/05/2013, Johannes Hofmann <johannes.hofm...@gmx.de> wrote:
> > -           switch(select(MAX(xfd, cmdfd)+1, &rfd, NULL, NULL, tv) < 0) {
> > +           switch(select(MAX(xfd, cmdfd)+1, &rfd, NULL, NULL, tv)) {
> >             case -1:
> >                     if(errno == EINTR)
> >                             continue;
> >
> 
> Why are we even using switch here?

That was an old regression when I added blink support. This was supposed
to handle the events and timeouts differently, but it came out that han‐
dling  both  in  a  common TIMEDIFF() way is more easier and keeps state
more easily.

Thanks for the hint.


Sincerely,

Christoph Lohmann


Reply via email to