Hi, Sorry for the long delay,
On Thu, Aug 30, 2007 at 07:42:51PM -0400, Alan Curry wrote: [...] > 2007-08-30 Alan Curry <[EMAIL PROTECTED]> > > * stream.c (sprintflags): New function. > (decode_poll): Rearrange so that arguments are decoded and printed on > syscall entry, except for revents which are now printed in the auxstr. The output timespec should be printed in the auxstr as well. > + if (syserror(tcp)) > + return 0; [...] > + if (!fds.revents) > + continue; [...] > + if (fds.fd < 0) { > + sprintf(str, "{fd=%d}", fds.fd); > + cumlen += strlen(str); > + if (cumlen < sizeof(outstr)) > + strcat(outstr, str); > + continue; > + } This fds.fd check is redundant: if syscall succeeded and "revents" is set, then "fd" should be printed unconditionally. I'll submit an updated change in 3 parts. -- ldv
pgpAFtPe046iw.pgp
Description: PGP signature