From de7cf8335ece0e6c54ceb1979970f47a032e1d22 Mon Sep 17 00:00:00 2001 From: jsalling Date: Mon, 23 Nov 2015 21:03:46 -0600 Subject: [PATCH] Transform plain macro into a function macro, UNITY_PRINT_EOL() This helps clarity and is more obvious, it looks like a print function --- extras/fixture/src/unity_fixture.c | 10 +++++----- src/unity.c | 10 +++++----- src/unity_internals.h | 2 +- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/extras/fixture/src/unity_fixture.c b/extras/fixture/src/unity_fixture.c index b6f8a65..45a2c02 100644 --- a/extras/fixture/src/unity_fixture.c +++ b/extras/fixture/src/unity_fixture.c @@ -28,7 +28,7 @@ static void announceTestRun(unsigned int runNumber) UnityPrintNumber(runNumber+1); UnityPrint(" of "); UnityPrintNumber(UnityFixture.RepeatCount); - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); } int UnityMain(int argc, const char* argv[], void (*runAllTests)(void)) @@ -43,7 +43,7 @@ int UnityMain(int argc, const char* argv[], void (*runAllTests)(void)) UnityBegin(argv[0]); announceTestRun(r); runAllTests(); - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); UnityEnd(); } @@ -396,7 +396,7 @@ void UnityConcludeFixtureTest(void) { //if (UnityFixture.Verbose) //{ - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); //} Unity.TestIgnores++; } @@ -405,13 +405,13 @@ void UnityConcludeFixtureTest(void) if (UnityFixture.Verbose) { UnityPrint(" PASS"); - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); } } else if (Unity.CurrentTestFailed) { Unity.TestFailures++; - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); } Unity.CurrentTestFailed = 0; diff --git a/src/unity.c b/src/unity.c index 5e4018a..f49b191 100644 --- a/src/unity.c +++ b/src/unity.c @@ -355,7 +355,7 @@ void UnityConcludeTest(void) Unity.CurrentTestFailed = 0; Unity.CurrentTestIgnored = 0; - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); } //----------------------------------------------- @@ -1272,16 +1272,16 @@ void UnityBegin(const char* filename) //----------------------------------------------- int UnityEnd(void) { - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); UnityPrint(UnityStrBreaker); - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); UnityPrintNumber((_U_SINT)(Unity.NumberOfTests)); UnityPrint(UnityStrResultsTests); UnityPrintNumber((_U_SINT)(Unity.TestFailures)); UnityPrint(UnityStrResultsFailures); UnityPrintNumber((_U_SINT)(Unity.TestIgnores)); UnityPrint(UnityStrResultsIgnored); - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); if (Unity.TestFailures == 0U) { UnityPrintOk(); @@ -1290,7 +1290,7 @@ int UnityEnd(void) { UnityPrintFail(); } - UNITY_PRINT_EOL; + UNITY_PRINT_EOL(); UNITY_OUTPUT_COMPLETE(); return (int)(Unity.TestFailures); } diff --git a/src/unity_internals.h b/src/unity_internals.h index 231799a..26d18e3 100644 --- a/src/unity_internals.h +++ b/src/unity_internals.h @@ -292,7 +292,7 @@ extern int UNITY_OUTPUT_CHAR(int); #endif #ifndef UNITY_PRINT_EOL -#define UNITY_PRINT_EOL UNITY_OUTPUT_CHAR('\n') +#define UNITY_PRINT_EOL() UNITY_OUTPUT_CHAR('\n') #endif #ifndef UNITY_OUTPUT_START -- GitLab