Whoops, you're right there. I think it is a leftover, I don't think it
is the right thing to do.


On Fri, May 29, 2015 at 01:32:12PM +0200, S??bastien Marie wrote:
> On Fri, May 29, 2015 at 12:00:36PM +0100, Nicholas Marriott wrote:
> > Hi
> > 
> > Why do you want to set O_NONBLOCK at all?
> > 
> > >           } else {
> > >                   fd = open(argv[idx], O_RDONLY|O_NONBLOCK);
> > >                   if (fd == -1 && (errno == ENFILE || errno == EMFILE))
> 
> Because of open is used with O_NONBLOCK ?
> 
> But, if it isn't need it could be removed from my patch.
> -- 
> S??bastien Marie

Reply via email to