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

Attachment: pgpAFtPe046iw.pgp
Description: PGP signature

Reply via email to