From bc9e12ea1a3fa391f691e9ddd9dae4501f1aadf3 Mon Sep 17 00:00:00 2001 From: Mark VanderVoord Date: Fri, 19 Jun 2015 15:16:10 -0400 Subject: [PATCH] - Trying a very verbose method of incrementing the pointers - casting to a (void*) was giving warnings to some compilers about double casting - casting from a u8 to u16/32/64 was giving warnings about changing alignment requirements --- src/unity.c | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/unity.c b/src/unity.c index f4c3aa9..f083d5f 100644 --- a/src/unity.c +++ b/src/unity.c @@ -514,8 +514,8 @@ void UnityAssertEqualIntArray(UNITY_PTR_ATTRIBUTE const void* expected, const UNITY_DISPLAY_STYLE_T style) { _UU32 elements = num_elements; - UNITY_PTR_ATTRIBUTE const _US8* ptr_exp = (UNITY_PTR_ATTRIBUTE const _US8*)expected; - UNITY_PTR_ATTRIBUTE const _US8* ptr_act = (UNITY_PTR_ATTRIBUTE const _US8*)actual; + UNITY_PTR_ATTRIBUTE const void* ptr_exp = (UNITY_PTR_ATTRIBUTE const void*)expected; + UNITY_PTR_ATTRIBUTE const void* ptr_act = (UNITY_PTR_ATTRIBUTE const void*)actual; UNITY_SKIP_EXECUTION; @@ -540,20 +540,20 @@ void UnityAssertEqualIntArray(UNITY_PTR_ATTRIBUTE const void* expected, case UNITY_DISPLAY_STYLE_UINT8: while (elements--) { - if (*ptr_exp != *ptr_act) + if (*(UNITY_PTR_ATTRIBUTE const _US8*)ptr_exp != *(UNITY_PTR_ATTRIBUTE const _US8*)ptr_act) { UnityTestResultsFailBegin(lineNumber); UnityPrint(UnityStrElement); UnityPrintNumberByStyle((num_elements - elements - 1), UNITY_DISPLAY_STYLE_UINT); UnityPrint(UnityStrExpected); - UnityPrintNumberByStyle(*ptr_exp, style); + UnityPrintNumberByStyle(*(UNITY_PTR_ATTRIBUTE const _US8*)ptr_exp, style); UnityPrint(UnityStrWas); - UnityPrintNumberByStyle(*ptr_act, style); + UnityPrintNumberByStyle(*(UNITY_PTR_ATTRIBUTE const _US8*)ptr_act, style); UnityAddMsgIfSpecified(msg); UNITY_FAIL_AND_BAIL; } - ptr_exp += 1; - ptr_act += 1; + ptr_exp = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_exp + 1); + ptr_act = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_act + 1); } break; case UNITY_DISPLAY_STYLE_HEX16: @@ -573,8 +573,8 @@ void UnityAssertEqualIntArray(UNITY_PTR_ATTRIBUTE const void* expected, UnityAddMsgIfSpecified(msg); UNITY_FAIL_AND_BAIL; } - ptr_exp += 2; - ptr_act += 2; + ptr_exp = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_exp + 2); + ptr_act = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_act + 2); } break; #ifdef UNITY_SUPPORT_64 @@ -595,8 +595,8 @@ void UnityAssertEqualIntArray(UNITY_PTR_ATTRIBUTE const void* expected, UnityAddMsgIfSpecified(msg); UNITY_FAIL_AND_BAIL; } - ptr_exp += 8; - ptr_act += 8; + ptr_exp = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_exp + 8); + ptr_act = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_act + 8); } break; #endif @@ -615,8 +615,8 @@ void UnityAssertEqualIntArray(UNITY_PTR_ATTRIBUTE const void* expected, UnityAddMsgIfSpecified(msg); UNITY_FAIL_AND_BAIL; } - ptr_exp += 4; - ptr_act += 4; + ptr_exp = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_exp + 4); + ptr_act = (UNITY_PTR_ATTRIBUTE const void*)((_UP)ptr_act + 4); } break; }