Fix errors reported by KWStyle in tests folder #117
Conversation
| @@ -72,4 +72,3 @@ Suite *make_log_internal_suite(void) | |||
|
|
|||
| return s; | |||
| } | |||
|
|
|||
There was a problem hiding this comment.
I like the extra line at the end of files, though. And, technically all non-empty source files must end in a new line according to the C standard: http://gcc.gnu.org/ml/gcc/2003-11/msg01568.html
There was a problem hiding this comment.
Currently (without my commit) there are 2 empty lines. The view of GitHub hides the last one. Check the file with cat.
| @@ -91,194 +93,356 @@ static master_test_t master_tests[] = { | |||
| { "Simple Tests", "test_ck_abort", CK_FAILURE, CK_MSG_TEXT, "Failed" }, | |||
| { "Simple Tests", "test_ck_abort_msg", CK_FAILURE, CK_MSG_TEXT, "Failure expected" }, | |||
| { "Simple Tests", "test_ck_abort_msg_null", CK_FAILURE, CK_MSG_TEXT, "Failed" }, | |||
| { "Simple Tests", "test_ck_assert", CK_FAILURE, CK_MSG_TEXT, "Assertion 'x == y' failed" }, | |||
There was a problem hiding this comment.
I know that these are really long lines, but I do prefer each entry to be on one line. At least for this table, that is. Some of the other long line examples that you have edited are fine.
| @@ -611,8 +779,8 @@ START_TEST(test_check_failure_ftypes) | |||
| tr = tr_fail_array[i - passed]; | |||
| ck_assert_msg(tr != NULL, NULL); | |||
| ck_assert_msg(master_tests[i].failure_type == tr_rtype(tr), | |||
| "Failure type wrong for test %d:%s:%s", | |||
There was a problem hiding this comment.
I'm confused what is going on here. Are the tabs being replaced with spaces, or only some of the tabs?
There was a problem hiding this comment.
GNU Emacs (default) auto-indentation, I will replace tabs by spaces.
#100