Re: [dev] [st] minor fix

From: Christoph Lohmann <20h_AT_r-36.net>
Date: Mon, 06 May 2013 19:48:37 +0200

Greetings.

On Mon, 06 May 2013 19:48:37 +0200 Strake <strake888_AT_gmail.com> wrote:
> On 06/05/2013, Johannes Hofmann <johannes.hofmann_AT_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
Received on Mon May 06 2013 - 19:48:37 CEST