> > > + if (!ret) > > > + timeout = 1; > > > + else > > > + timeout = 0; > > > > > > nit: the following might be cleaner > > > > ret = !!timeout; > > > > Other way around, perhaps, > > timeout = !ret; I saw that but didn't want to modify this patch. But yes, this is easy enough and looks a lot better.