Re: [PATCH 5/8] xfstests: fix compile warnings in iopat.c

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Wed, Jan 20, 2010 at 02:38:05PM +1100, Dave Chinner wrote:
> Printf format warnings and return value warnings need fixing.
> 
> Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>
> ---
>  src/iopat.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/src/iopat.c b/src/iopat.c
> index c06071f..f34de19 100644
> --- a/src/iopat.c
> +++ b/src/iopat.c
> @@ -47,12 +47,13 @@ int main(int argc, char **argv)
>     read(fd, &x, 1048576);
>     for (i = 0; i < 131072; i++) {
>        if (x[i] != i) {
> -         printf("error: %d %d %d\n",i,8*i,x[i]);
> +         printf("error: %d %d %lld\n", i ,8 * i, (long long)x[i]);
>        exit(1);

Instead of the casting we could also use the C99 macro for the uin64_t
format.  But either way good enough for the tests,


Reviewed-by: Christoph Hellwig <hch@xxxxxx>