[Top][All Lists]

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

Re: Questions about patches for hurdselect.c

From: Samuel Thibault
Subject: Re: Questions about patches for hurdselect.c
Date: Sun, 13 Jan 2013 21:31:38 +0100
User-agent: Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30)

Svante Signell, le Sun 13 Jan 2013 21:18:07 +0100, a écrit :
> > - In the poll case, on error on one descriptor, the loop shall mark
> > POLLERR and continue, not fail altogether.
> > 
> > Is that right?
> The current code errors out if it finds one FD not OK, which is a bug.
> Irrespective of how many FDs there are, no checks are made of the input
> data given.

Sure, but what you refer to is only one part of the story. EBADF is one
of the "error"s I mention above.  Errors returned by translators also
need the same fix.

> > > - Hopefully the code is more readable now, without causing regressions.
> > 
> > But it duplicates things. As I told you, we might still keep the code
> > as-is, and just apply your poll fixes over it. The 3-split still being
> > useful for reviewing.
> Well, slitting into three parts made the poll case faster

Uh? Why?

> the poll case _is_ different from select, see the POSIX specs.

In a very limited way only.


reply via email to

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