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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxxxx>
-Chris

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux