ERROR: trailing whitespace #36: FILE: .gitignore:74: +!test/cmocka/include $ CHECK: Macro argument reuse 'ptr' - possible side-effects? #70: FILE: test/cmocka/include/test_group_generator.h:2: +#define setup_alloc(ptr, type, size, offset) do {\ + if (ptr)\ + free(ptr);\ + ptr = malloc(sizeof(type) * (size) + offset);\ + if (ptr == NULL)\ + return -1;\ +} while (0) WARNING: Macros with flow control statements should be avoided #70: FILE: test/cmocka/include/test_group_generator.h:2: +#define setup_alloc(ptr, type, size, offset) do {\ + if (ptr)\ + free(ptr);\ + ptr = malloc(sizeof(type) * (size) + offset);\ + if (ptr == NULL)\ + return -1;\ +} while (0) CHECK: Comparison to NULL could be written "!ptr" #74: FILE: test/cmocka/include/test_group_generator.h:6: + if (ptr == NULL)\ CHECK: Macro argument reuse 'result' - possible side-effects? #78: FILE: test/cmocka/include/test_group_generator.h:10: +#define setup_part(result, setup_func) do {\ + result |= setup_func;\ + if (result)\ + return result;\ +} while (0) WARNING: Macros with flow control statements should be avoided #78: FILE: test/cmocka/include/test_group_generator.h:10: +#define setup_part(result, setup_func) do {\ + result |= setup_func;\ + if (result)\ + return result;\ +} while (0) CHECK: Macro argument reuse 'prefix_setup' - possible side-effects? #113: FILE: test/cmocka/include/test_group_generator.h:45: +#define gen_test_with_prefix(prefix_check, prefix_setup, name, ...) \ +static void gen_test_concat_func(prefix_setup, name, __VA_ARGS__) \ +(void **state) {\ + if (!gen_test_concat_flag(prefix_setup, __VA_ARGS__)) {\ + if (_setup(state, __VA_ARGS__))\ + exit(1);\ + gen_test_concat_base(prefix_setup)(state);\ + ++gen_test_concat_flag(prefix_setup, __VA_ARGS__);\ + } \ + gen_test_concat(prefix_check, name)(state);\ +} CHECK: Macro argument reuse 'name' - possible side-effects? #113: FILE: test/cmocka/include/test_group_generator.h:45: +#define gen_test_with_prefix(prefix_check, prefix_setup, name, ...) \ +static void gen_test_concat_func(prefix_setup, name, __VA_ARGS__) \ +(void **state) {\ + if (!gen_test_concat_flag(prefix_setup, __VA_ARGS__)) {\ + if (_setup(state, __VA_ARGS__))\ + exit(1);\ + gen_test_concat_base(prefix_setup)(state);\ + ++gen_test_concat_flag(prefix_setup, __VA_ARGS__);\ + } \ + gen_test_concat(prefix_check, name)(state);\ +} WARNING: macros should not use a trailing semicolon #138: FILE: test/cmocka/include/test_group_generator.h:70: +#define flg_test_group(prefix, ...)\ + static int gen_test_concat_flag(prefix, __VA_ARGS__); CHECK: Alignment should match open parenthesis #419: FILE: test/cmocka/src/lib/lib/strcheck.c:58: +static int setup_change(void **state, + const size_t change_beg, const size_t change_end) CHECK: Alignment should match open parenthesis #455: FILE: test/cmocka/src/lib/lib/strcheck.c:94: +static int _setup(void **state, size_t data_len, + size_t change_beg, size_t change_end) CHECK: Lines should not end with a '(' #519: FILE: test/cmocka/src/lib/lib/strcheck.c:158: + return check_arrs( CHECK: Lines should not end with a '(' #528: FILE: test/cmocka/src/lib/lib/strcheck.c:167: + return check_arrs( CHECK: Lines should not end with a '(' #543: FILE: test/cmocka/src/lib/lib/strcheck.c:182: + assert_int_equal( CHECK: Lines should not end with a '(' #544: FILE: test/cmocka/src/lib/lib/strcheck.c:183: + check_arrs( CHECK: Lines should not end with a '(' #559: FILE: test/cmocka/src/lib/lib/strcheck.c:198: + assert_int_equal( CHECK: Lines should not end with a '(' #560: FILE: test/cmocka/src/lib/lib/strcheck.c:199: + check_arrs( CHECK: Lines should not end with a '(' #573: FILE: test/cmocka/src/lib/lib/strcheck.c:212: + assert_int_equal( CHECK: Lines should not end with a '(' #586: FILE: test/cmocka/src/lib/lib/strcheck.c:225: + test_data->func_ret = memset( CHECK: Lines should not end with a '(' #599: FILE: test/cmocka/src/lib/lib/strcheck.c:238: + bzero( CHECK: Lines should not end with a '(' #617: FILE: test/cmocka/src/lib/lib/strcheck.c:256: + test_data->func_ret = memcpy( CHECK: Alignment should match open parenthesis #632: FILE: test/cmocka/src/lib/lib/strcheck.c:271: + assert_int_equal(expected, + check_test_arrs()); CHECK: Alignment should match open parenthesis #640: FILE: test/cmocka/src/lib/lib/strcheck.c:279: + assert_int_equal(expected, + check_arrs(test_data->before + test_data_change->beg, CHECK: Alignment should match open parenthesis #641: FILE: test/cmocka/src/lib/lib/strcheck.c:280: + check_arrs(test_data->before + test_data_change->beg, + test_data->func_ret, CHECK: Alignment should match open parenthesis #648: FILE: test/cmocka/src/lib/lib/strcheck.c:287: + assert_int_equal(-1, + check_arr(test_data->after + test_data_change->beg, CHECK: Alignment should match open parenthesis #649: FILE: test/cmocka/src/lib/lib/strcheck.c:288: + check_arr(test_data->after + test_data_change->beg, + test_data_change->len, CHECK: Alignment should match open parenthesis #656: FILE: test/cmocka/src/lib/lib/strcheck.c:295: + assert_int_equal(-1, + check_arr(test_data->func_ret, CHECK: Alignment should match open parenthesis #657: FILE: test/cmocka/src/lib/lib/strcheck.c:296: + check_arr(test_data->func_ret, + test_data_change->len, CHECK: Alignment should match open parenthesis #664: FILE: test/cmocka/src/lib/lib/strcheck.c:303: + assert_int_equal(-1, + check_test_arrs_with_offset(test_data_change->end)); CHECK: Alignment should match open parenthesis #670: FILE: test/cmocka/src/lib/lib/strcheck.c:309: + assert_int_equal(-1, + check_arrs(test_data->func_ret + test_data_change->len, CHECK: Alignment should match open parenthesis #671: FILE: test/cmocka/src/lib/lib/strcheck.c:310: + check_arrs(test_data->func_ret + test_data_change->len, + test_data->before + test_data_change->end, ERROR: Macros with complex values should be enclosed in parentheses #684: FILE: test/cmocka/src/lib/lib/strcheck.c:323: +#define bind_test_group_ref_ret(bind, ...)\ + bind(ref_beg, __VA_ARGS__)\ + bind(ret_beg, __VA_ARGS__)\ + bind(ref_mid, __VA_ARGS__)\ + bind(ret_mid, __VA_ARGS__)\ + bind(ref_end, __VA_ARGS__)\ + bind(ret_end, __VA_ARGS__) CHECK: Macro argument reuse 'bind' - possible side-effects? #684: FILE: test/cmocka/src/lib/lib/strcheck.c:323: +#define bind_test_group_ref_ret(bind, ...)\ + bind(ref_beg, __VA_ARGS__)\ + bind(ret_beg, __VA_ARGS__)\ + bind(ref_mid, __VA_ARGS__)\ + bind(ret_mid, __VA_ARGS__)\ + bind(ref_end, __VA_ARGS__)\ + bind(ret_end, __VA_ARGS__) ERROR: Macros with complex values should be enclosed in parentheses #692: FILE: test/cmocka/src/lib/lib/strcheck.c:331: +#define bind_test_group_ref(bind, ...)\ + bind(ref_beg, __VA_ARGS__)\ + bind(ref_mid, __VA_ARGS__)\ + bind(ref_end, __VA_ARGS__) CHECK: Macro argument reuse 'bind' - possible side-effects? #692: FILE: test/cmocka/src/lib/lib/strcheck.c:331: +#define bind_test_group_ref(bind, ...)\ + bind(ref_beg, __VA_ARGS__)\ + bind(ref_mid, __VA_ARGS__)\ + bind(ref_end, __VA_ARGS__) ERROR: Macros with complex values should be enclosed in parentheses #714: FILE: test/cmocka/src/lib/lib/strcheck.c:353: +#define run_test_group_action(action, bind_macro) \ + action(bind_macro, 3, 2, 2) /*change none characters*/\ + action(bind_macro, 5, 0, 1) /*change first character */\ + action(bind_macro, 7, 0, 7) /*change all characters*/\ + action(bind_macro, 9, 3, 6) /*change some characters*/\ + action(bind_macro, 2048, 512, 1024) /*change some character - long*/ CHECK: Macro argument reuse 'action' - possible side-effects? #714: FILE: test/cmocka/src/lib/lib/strcheck.c:353: +#define run_test_group_action(action, bind_macro) \ + action(bind_macro, 3, 2, 2) /*change none characters*/\ + action(bind_macro, 5, 0, 1) /*change first character */\ + action(bind_macro, 7, 0, 7) /*change all characters*/\ + action(bind_macro, 9, 3, 6) /*change some characters*/\ + action(bind_macro, 2048, 512, 1024) /*change some character - long*/ CHECK: Macro argument reuse 'bind_macro' - possible side-effects? #714: FILE: test/cmocka/src/lib/lib/strcheck.c:353: +#define run_test_group_action(action, bind_macro) \ + action(bind_macro, 3, 2, 2) /*change none characters*/\ + action(bind_macro, 5, 0, 1) /*change first character */\ + action(bind_macro, 7, 0, 7) /*change all characters*/\ + action(bind_macro, 9, 3, 6) /*change some characters*/\ + action(bind_macro, 2048, 512, 1024) /*change some character - long*/ CHECK: Avoid CamelCase: #733: FILE: test/cmocka/src/lib/lib/strcheck.c:372: + const struct CMUnitTest tests[] = { total: 4 errors, 3 warnings, 32 checks, 498 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. NOTE: Whitespace errors detected. You may wish to use scripts/cleanpatch or scripts/cleanfile 0001-lib-strcheck-added-unit-tests-for-lib-memset-lib-mem.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.