diff options
author | Matt Roper <matthew.d.roper@intel.com> | 2015-03-05 15:01:00 -0800 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-03-06 18:06:10 +0100 |
commit | 07be8fec156ba8405f2aafa3a37fe29d06893f05 (patch) | |
tree | 0591bf1aca05c2c8f5f89bb0400d948e8a2d0fe6 /tests/kms_cursor_crc.c | |
parent | 14a3d88a0967718124fc3b6c111a877203da025f (diff) |
igt.cocci: Replace igt_assert() with igt_assert_CMP() where possible
The integer comparison macros give us better error output by including
the actual values that failed the comparison.
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests/kms_cursor_crc.c')
-rw-r--r-- | tests/kms_cursor_crc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index 64fea122..94e2b4a7 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -159,7 +159,7 @@ static void do_fail_test(data_t *data, int x, int y, int expect) cursor_disable(data); igt_display_commit(display); - igt_assert(ret == expect); + igt_assert_eq(ret, expect); } static void do_test(data_t *data, @@ -429,12 +429,12 @@ static void test_cursor_size(data_t *data) /* Hardware test loop */ cursor_enable(data); ret = drmModeMoveCursor(data->drm_fd, data->output->config.crtc->crtc_id, 0, 0); - igt_assert(ret == 0); + igt_assert_eq(ret, 0); for (i = 0, size = cursor_max_size; size >= 64; size /= 2, i++) { /* Change size in flight: */ ret = drmModeSetCursor(data->drm_fd, data->output->config.crtc->crtc_id, data->fb.gem_handle, size, size); - igt_assert(ret == 0); + igt_assert_eq(ret, 0); igt_wait_for_vblank(data->drm_fd, data->pipe); igt_pipe_crc_collect_crc(pipe_crc, &crc[i]); } |