On Mon, Aug 29, 2016 at 06:12:15PM +0300, Eugene Syromyatnikov wrote:
> Several changes have been performed:
>  * alternate form is used for timeout value since it is pointer half
>    of the time. And half of the time it is not, which can be confusing
>    without number base prefix.
>  * uaddr is pointer across all existing operations, so it's rather expected
>    that it would be the same in possible future operations.
>  * val3 numeric base is also different in different commands, so it's
>    better to use alternate form for it too.
> 
> * futex.c (SYS_FUNC(futex)): update formatting in unknown command case.
> ---
>  futex.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/futex.c b/futex.c
> index 44b90e0..6bd739f 100644
> --- a/futex.c
> +++ b/futex.c
> @@ -107,7 +107,10 @@ SYS_FUNC(futex)
>       case FUTEX_TRYLOCK_PI:
>               break;
>       default:
> -             tprintf(", %lx, %lx, %x", timeout, uaddr2, val3);
> +             tprintf(", %#lx", timeout);
> +             tprints(", ");
> +             printaddr(uaddr2);
> +             tprintf(", %#x", val3);
>               break;
>       }
>  

Looks fine.


-- 
ldv

Attachment: pgpOxAbDGf2fp.pgp
Description: PGP signature

------------------------------------------------------------------------------
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to