diff options
| author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2014-04-25 13:38:31 +0300 |
|---|---|---|
| committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2014-04-25 16:18:19 +0300 |
| commit | f688a56ee8285f6ebc416d5038eded73f04602bb (patch) | |
| tree | 83b4d11e983d8ed2bae68b1aefdee40b63fd3590 /tests/kms_cursor_crc.c | |
| parent | fb2ccb1ade605e7c55da2ee82c9cd41d98270fa5 (diff) | |
kms_cursor_crc: Replace snprintf()+%s with just %d
There's no point in first printing the int to temp string using %d
and then printing that out with %s. Just stick the %d into the final
string.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Diffstat (limited to 'tests/kms_cursor_crc.c')
| -rw-r--r-- | tests/kms_cursor_crc.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index d2cb7e37..6934d6b6 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -348,23 +348,21 @@ static void create_cursor_fb(data_t *data, int cur_w, int cur_h) static void run_test_generic(data_t *data, int cursor_max_size) { int cursor_size; - char c_size[5]; for (cursor_size = 64; cursor_size <= 256; cursor_size *= 2) { igt_require(cursor_max_size >= cursor_size); - sprintf(c_size, "%d", cursor_size); igt_fixture create_cursor_fb(data, cursor_size, cursor_size); /* Using created cursor FBs to test cursor support */ - igt_subtest_f("cursor-%s-onscreen", c_size) + igt_subtest_f("cursor-%d-onscreen", cursor_size) run_test(data, test_crc_onscreen, cursor_size, cursor_size); - igt_subtest_f("cursor-%s-offscreen", c_size) + igt_subtest_f("cursor-%d-offscreen", cursor_size) run_test(data, test_crc_offscreen, cursor_size, cursor_size); - igt_subtest_f("cursor-%s-sliding", c_size) + igt_subtest_f("cursor-%d-sliding", cursor_size) run_test(data, test_crc_sliding, cursor_size, cursor_size); - igt_subtest_f("cursor-%s-random", c_size) + igt_subtest_f("cursor-%d-random", cursor_size) run_test(data, test_crc_random, cursor_size, cursor_size); igt_fixture |