Re: [PATCH v2] rt-tests: printf format compile warning

[Clark Williams <williams@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1157 bytes --]

On Thu, 03 May 2012 14:13:56 -0700
Darren Hart <dvhart@linux.intel.com> wrote:

> 
> 
> On 05/03/2012 12:48 PM, Frank Rowand wrote:
> > 
> > V2: use type casting instead of ugly constant in format string
> > 
> > 
> > Fix printf format string to fix compile warning for ARM 32 bit target.
> > 
> 
> Oh much better. :-)
> 
> > Signed-off-by: Frank Rowand <frank.rowand@am.sony.com>
> > ---
> >  src/cyclictest/cyclictest.c |    2 	1 +	1 -	0 !
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > Index: b/src/cyclictest/cyclictest.c
> > ===================================================================
> > --- a/src/cyclictest/cyclictest.c
> > +++ b/src/cyclictest/cyclictest.c
> > @@ -1575,7 +1575,7 @@ int main(int argc, char **argv)
> >  		print_tids(parameters, num_threads);
> >  		if (break_thread_id) {
> >  			printf("# Break thread: %d\n", break_thread_id);
> > -			printf("# Break value: %lu\n", break_thread_value);
> > +			printf("# Break value: %llu\n", (unsigned long long)break_thread_value);
> >  		}
> >  	}
> >  	
> > 
> 
>

Yes, I like this better. I pulled the original and applied this.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 198 bytes --]