On Thu, 14 Mar 2013, Pawel Jakub Dawidek wrote: > Right. Your patch assumes EWOULDBLOCK is equal to EAGAIN, which is true > on FreeBSD, but is not portable. Also in case pidptr is NULL you compare > errno three times instead of just one (not a big deal of course, just > something that could be done a bit more optimal:)). You're right. OTOH, wouldn't modifying pidfile_read to return EAGAIN and not EWOULDBLOCK totally be worth saving that 2 lines in the patch? ;) erdgeistReceived on Thu Mar 14 2013 - 08:33:39 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:35 UTC