diff --git a/extras/fixture/src/unity_fixture.c b/extras/fixture/src/unity_fixture.c index 67e7b9d7e563dbbc4e32ff5bf28b2c3678c2d135..90314af48f6ac5f4be57276ef622f14f2705c567 100644 --- a/extras/fixture/src/unity_fixture.c +++ b/extras/fixture/src/unity_fixture.c @@ -23,9 +23,9 @@ void tearDown(void) { /*does nothing*/ } static void announceTestRun(unsigned int runNumber) { UnityPrint("Unity test run "); - UnityPrintNumber(runNumber+1); + UnityPrintNumberUnsigned(runNumber+1); UnityPrint(" of "); - UnityPrintNumber(UnityFixture.RepeatCount); + UnityPrintNumberUnsigned(UnityFixture.RepeatCount); UNITY_PRINT_EOL(); } @@ -45,7 +45,7 @@ int UnityMain(int argc, const char* argv[], void (*runAllTests)(void)) UnityEnd(); } - return Unity.TestFailures; + return (int)Unity.TestFailures; } static int selected(const char* filter, const char* name) @@ -71,7 +71,7 @@ void UnityTestRunner(unityfunction* setup, const char* printableName, const char* group, const char* name, - const char* file, int line) + const char* file, unsigned int line) { if (testSelected(name) && groupSelected(group)) { diff --git a/extras/fixture/src/unity_fixture_internals.h b/extras/fixture/src/unity_fixture_internals.h index cd27ccfa6cda389b80dfd4e671d266998d4dec8c..1fdbb9893dac8d7275b2453e195598280ca974d7 100644 --- a/extras/fixture/src/unity_fixture_internals.h +++ b/extras/fixture/src/unity_fixture_internals.h @@ -23,7 +23,7 @@ void UnityTestRunner(unityfunction* setup, const char* printableName, const char* group, const char* name, - const char* file, int line); + const char* file, unsigned int line); void UnityIgnoreTest(const char* printableName, const char* group, const char* name); void UnityMalloc_StartTest(void); diff --git a/extras/fixture/test/Makefile b/extras/fixture/test/Makefile index 66fc58be3be15d5441f272d5c09c9c77fd0d3075..f3f7ab1bc3bc8f70e2d912b384dcb3c2f5dde065 100644 --- a/extras/fixture/test/Makefile +++ b/extras/fixture/test/Makefile @@ -46,7 +46,7 @@ clean: # These extended flags DO get included before any target build runs CFLAGS += -Wbad-function-cast CFLAGS += -Wcast-qual -#CFLAGS += -Wconversion +CFLAGS += -Wconversion CFLAGS += -Wformat=2 CFLAGS += -Wmissing-prototypes CFLAGS += -Wold-style-definition diff --git a/extras/fixture/test/unity_fixture_Test.c b/extras/fixture/test/unity_fixture_Test.c index 2a16e2d2a0cdca6b219a1f541ffd473568265a57..ffb5c2a0aff26b76d5182746b960903bfa375b9d 100644 --- a/extras/fixture/test/unity_fixture_Test.c +++ b/extras/fixture/test/unity_fixture_Test.c @@ -144,7 +144,7 @@ TEST(UnityFixture, FreeNULLSafety) TEST_GROUP(UnityCommandOptions); int savedVerbose; -int savedRepeat; +unsigned int savedRepeat; const char* savedName; const char* savedGroup; diff --git a/extras/fixture/test/unity_output_Spy.c b/extras/fixture/test/unity_output_Spy.c index 36bcf3751ce150915009875d7120edfb952a7442..97c952348c6bd1bebe22324f62b5ec7e0c47730c 100644 --- a/extras/fixture/test/unity_output_Spy.c +++ b/extras/fixture/test/unity_output_Spy.c @@ -22,7 +22,7 @@ void UnityOutputCharSpy_Create(int s) size = s; count = 0; spy_enable = 0; - buffer = malloc(size); + buffer = malloc((size_t)size); TEST_ASSERT_NOT_NULL_MESSAGE(buffer, "Internal malloc failed in Spy Create():" __FILE__); memset(buffer, 0, size); } @@ -38,7 +38,7 @@ int UnityOutputCharSpy_OutputChar(int c) if (spy_enable) { if (count < (size-1)) - buffer[count++] = c; + buffer[count++] = (char)c; } else {