From f688a56ee8285f6ebc416d5038eded73f04602bb Mon Sep 17 00:00:00 2001 From: Ville Syrjälä Date: Fri, 25 Apr 2014 13:38:31 +0300 Subject: kms_cursor_crc: Replace snprintf()+%s with just %d MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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ä --- tests/kms_cursor_crc.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'tests/kms_cursor_crc.c') 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 -- cgit v1.2.3