[Intel-gfx] [PATCH 1/2 i-g-t v2] lib: Add 64-bit version of igt_assert_cmp

Michel Thierry michel.thierry at intel.com
Tue Jun 30 06:56:59 PDT 2015


On 6/30/2015 2:49 PM, Chris Wilson wrote:
> On Tue, Jun 30, 2015 at 02:41:08PM +0100, Michel Thierry wrote:
>> igt_assert_cmp64 and its derivatives:
>>
>> +#define INTDECFORMAT "%d"
>> +#define UINTHEXFORMAT "%#x"
>> +#define DOUBLEDECFORMAT "%#lf"
>> +#define LONGHEXFORMAT "%#llx"
>
> Any particular reason? Passing in "%d" or %d would have worked just as
> well, right?

Not really, these defines are not needed. The format specifier can be 
passed directly.

>
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> -Chris
>


More information about the Intel-gfx mailing list