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