Merge pull request #254 from jsalling/cleanup/clang-format

Formatting source code to existing style
This commit is contained in:
Mark VanderVoord
2017-01-19 06:56:35 -05:00
committed by GitHub
2 changed files with 71 additions and 71 deletions

View File

@@ -5,9 +5,9 @@
* [Released under MIT License. Please refer to license.txt for details] * [Released under MIT License. Please refer to license.txt for details]
* ========================================== */ * ========================================== */
#include <string.h>
#include "unity_fixture.h" #include "unity_fixture.h"
#include "unity_internals.h" #include "unity_internals.h"
#include <string.h>
struct UNITY_FIXTURE_T UnityFixture; struct UNITY_FIXTURE_T UnityFixture;
@@ -71,7 +71,8 @@ void UnityTestRunner(unityfunction* setup,
const char* printableName, const char* printableName,
const char* group, const char* group,
const char* name, const char* name,
const char* file, unsigned int line) const char* file,
unsigned int line)
{ {
if (testSelected(name) && groupSelected(group)) if (testSelected(name) && groupSelected(group))
{ {
@@ -198,7 +199,7 @@ void* unity_malloc(size_t size)
} }
else else
{ {
guard = (Guard*) &unity_heap[heap_index]; guard = (Guard*)&unity_heap[heap_index];
heap_index += total_size; heap_index += total_size;
} }
#else #else
@@ -290,7 +291,7 @@ void* unity_realloc(void* oldMem, size_t size)
if (oldMem == unity_heap + heap_index - guard->size - sizeof(end) && if (oldMem == unity_heap + heap_index - guard->size - sizeof(end) &&
heap_index + size - guard->size <= UNITY_INTERNAL_HEAP_SIZE_BYTES) heap_index + size - guard->size <= UNITY_INTERNAL_HEAP_SIZE_BYTES)
{ {
release_memory(oldMem); /* Not thread-safe, like unity_heap generally */ release_memory(oldMem); /* Not thread-safe, like unity_heap generally */
return unity_malloc(size); /* No memcpy since data is in place */ return unity_malloc(size); /* No memcpy since data is in place */
} }
#endif #endif
@@ -395,7 +396,9 @@ int UnityGetCommandLineOptions(int argc, const char* argv[])
i++; i++;
} }
} }
} else { }
else
{
/* ignore unknown parameter */ /* ignore unknown parameter */
i++; i++;
} }

View File

@@ -180,8 +180,7 @@ void UnityPrintNumberUnsigned(const UNITY_UINT number)
{ {
UNITY_OUTPUT_CHAR((char)('0' + (number / divisor % 10))); UNITY_OUTPUT_CHAR((char)('0' + (number / divisor % 10)));
divisor /= 10; divisor /= 10;
} } while (divisor > 0);
while (divisor > 0);
} }
/*-----------------------------------------------*/ /*-----------------------------------------------*/
@@ -401,7 +400,7 @@ static void UnityPrintExpectedAndActualStrings(const char* expected, const char*
} }
else else
{ {
UnityPrint(UnityStrNull); UnityPrint(UnityStrNull);
} }
UnityPrint(UnityStrWas); UnityPrint(UnityStrWas);
if (actual != NULL) if (actual != NULL)
@@ -412,12 +411,14 @@ static void UnityPrintExpectedAndActualStrings(const char* expected, const char*
} }
else else
{ {
UnityPrint(UnityStrNull); UnityPrint(UnityStrNull);
} }
} }
/*-----------------------------------------------*/ /*-----------------------------------------------*/
static void UnityPrintExpectedAndActualStringsLen(const char* expected, const char* actual, const UNITY_UINT32 length) static void UnityPrintExpectedAndActualStringsLen(const char* expected,
const char* actual,
const UNITY_UINT32 length)
{ {
UnityPrint(UnityStrExpected); UnityPrint(UnityStrExpected);
if (expected != NULL) if (expected != NULL)
@@ -428,7 +429,7 @@ static void UnityPrintExpectedAndActualStringsLen(const char* expected, const ch
} }
else else
{ {
UnityPrint(UnityStrNull); UnityPrint(UnityStrNull);
} }
UnityPrint(UnityStrWas); UnityPrint(UnityStrWas);
if (actual != NULL) if (actual != NULL)
@@ -439,12 +440,10 @@ static void UnityPrintExpectedAndActualStringsLen(const char* expected, const ch
} }
else else
{ {
UnityPrint(UnityStrNull); UnityPrint(UnityStrNull);
} }
} }
/*----------------------------------------------- /*-----------------------------------------------
* Assertion & Control Helpers * Assertion & Control Helpers
*-----------------------------------------------*/ *-----------------------------------------------*/
@@ -580,7 +579,7 @@ void UnityAssertEqualIntArray(UNITY_INTERNAL_PTR expected,
if (expect_val != actual_val) if (expect_val != actual_val)
{ {
if (style & UNITY_DISPLAY_RANGE_UINT && length < sizeof expect_val) if (style & UNITY_DISPLAY_RANGE_UINT && length < sizeof(expect_val))
{ /* For UINT, remove sign extension (padding 1's) from signed type casts above */ { /* For UINT, remove sign extension (padding 1's) from signed type casts above */
UNITY_INT mask = 1; UNITY_INT mask = 1;
mask = (mask << 8 * length) - 1; mask = (mask << 8 * length) - 1;
@@ -621,12 +620,11 @@ void UnityAssertEqualIntArray(UNITY_INTERNAL_PTR expected,
#ifndef UNITY_EXCLUDE_FLOAT_PRINT #ifndef UNITY_EXCLUDE_FLOAT_PRINT
#define UNITY_PRINT_EXPECTED_AND_ACTUAL_FLOAT(expected, actual) \ #define UNITY_PRINT_EXPECTED_AND_ACTUAL_FLOAT(expected, actual) \
do { \ { \
UnityPrint(UnityStrExpected); \ UnityPrint(UnityStrExpected); \
UnityPrintFloat(expected); \ UnityPrintFloat(expected); \
UnityPrint(UnityStrWas); \ UnityPrint(UnityStrWas); \
UnityPrintFloat(actual); \ UnityPrintFloat(actual); }
} while(0)
#else #else
#define UNITY_PRINT_EXPECTED_AND_ACTUAL_FLOAT(expected, actual) \ #define UNITY_PRINT_EXPECTED_AND_ACTUAL_FLOAT(expected, actual) \
UnityPrint(UnityStrDelta) UnityPrint(UnityStrDelta)
@@ -700,14 +698,14 @@ void UnityAssertFloatSpecial(const UNITY_FLOAT actual,
const UNITY_LINE_TYPE lineNumber, const UNITY_LINE_TYPE lineNumber,
const UNITY_FLOAT_TRAIT_T style) const UNITY_FLOAT_TRAIT_T style)
{ {
const char* trait_names[] = { UnityStrInf, UnityStrNegInf, UnityStrNaN, UnityStrDet }; const char* trait_names[] = {UnityStrInf, UnityStrNegInf, UnityStrNaN, UnityStrDet};
UNITY_INT should_be_trait = ((UNITY_INT)style & 1); UNITY_INT should_be_trait = ((UNITY_INT)style & 1);
UNITY_INT is_trait = !should_be_trait; UNITY_INT is_trait = !should_be_trait;
UNITY_INT trait_index = (UNITY_INT)(style >> 1); UNITY_INT trait_index = (UNITY_INT)(style >> 1);
RETURN_IF_FAIL_OR_IGNORE; RETURN_IF_FAIL_OR_IGNORE;
switch(style) switch (style)
{ {
case UNITY_FLOAT_IS_INF: case UNITY_FLOAT_IS_INF:
case UNITY_FLOAT_IS_NOT_INF: case UNITY_FLOAT_IS_NOT_INF:
@@ -826,14 +824,14 @@ void UnityAssertDoubleSpecial(const UNITY_DOUBLE actual,
const UNITY_LINE_TYPE lineNumber, const UNITY_LINE_TYPE lineNumber,
const UNITY_FLOAT_TRAIT_T style) const UNITY_FLOAT_TRAIT_T style)
{ {
const char* trait_names[] = { UnityStrInf, UnityStrNegInf, UnityStrNaN, UnityStrDet }; const char* trait_names[] = {UnityStrInf, UnityStrNegInf, UnityStrNaN, UnityStrDet};
UNITY_INT should_be_trait = ((UNITY_INT)style & 1); UNITY_INT should_be_trait = ((UNITY_INT)style & 1);
UNITY_INT is_trait = !should_be_trait; UNITY_INT is_trait = !should_be_trait;
UNITY_INT trait_index = (UNITY_INT)(style >> 1); UNITY_INT trait_index = (UNITY_INT)(style >> 1);
RETURN_IF_FAIL_OR_IGNORE; RETURN_IF_FAIL_OR_IGNORE;
switch(style) switch (style)
{ {
case UNITY_FLOAT_IS_INF: case UNITY_FLOAT_IS_INF:
case UNITY_FLOAT_IS_NOT_INF: case UNITY_FLOAT_IS_NOT_INF:
@@ -880,16 +878,15 @@ void UnityAssertDoubleSpecial(const UNITY_DOUBLE actual,
} }
} }
#endif /* not UNITY_EXCLUDE_DOUBLE */ #endif /* not UNITY_EXCLUDE_DOUBLE */
/*-----------------------------------------------*/ /*-----------------------------------------------*/
void UnityAssertNumbersWithin( const UNITY_UINT delta, void UnityAssertNumbersWithin(const UNITY_UINT delta,
const UNITY_INT expected, const UNITY_INT expected,
const UNITY_INT actual, const UNITY_INT actual,
const char* msg, const char* msg,
const UNITY_LINE_TYPE lineNumber, const UNITY_LINE_TYPE lineNumber,
const UNITY_DISPLAY_STYLE_T style) const UNITY_DISPLAY_STYLE_T style)
{ {
RETURN_IF_FAIL_OR_IGNORE; RETURN_IF_FAIL_OR_IGNORE;
@@ -954,19 +951,19 @@ void UnityAssertEqualString(const char* expected,
if (Unity.CurrentTestFailed) if (Unity.CurrentTestFailed)
{ {
UnityTestResultsFailBegin(lineNumber); UnityTestResultsFailBegin(lineNumber);
UnityPrintExpectedAndActualStrings(expected, actual); UnityPrintExpectedAndActualStrings(expected, actual);
UnityAddMsgIfSpecified(msg); UnityAddMsgIfSpecified(msg);
UNITY_FAIL_AND_BAIL; UNITY_FAIL_AND_BAIL;
} }
} }
/*-----------------------------------------------*/ /*-----------------------------------------------*/
void UnityAssertEqualStringLen(const char* expected, void UnityAssertEqualStringLen(const char* expected,
const char* actual, const char* actual,
const UNITY_UINT32 length, const UNITY_UINT32 length,
const char* msg, const char* msg,
const UNITY_LINE_TYPE lineNumber) const UNITY_LINE_TYPE lineNumber)
{ {
UNITY_UINT32 i; UNITY_UINT32 i;
@@ -994,20 +991,19 @@ void UnityAssertEqualStringLen(const char* expected,
if (Unity.CurrentTestFailed) if (Unity.CurrentTestFailed)
{ {
UnityTestResultsFailBegin(lineNumber); UnityTestResultsFailBegin(lineNumber);
UnityPrintExpectedAndActualStringsLen(expected, actual, length); UnityPrintExpectedAndActualStringsLen(expected, actual, length);
UnityAddMsgIfSpecified(msg); UnityAddMsgIfSpecified(msg);
UNITY_FAIL_AND_BAIL; UNITY_FAIL_AND_BAIL;
} }
} }
/*-----------------------------------------------*/ /*-----------------------------------------------*/
void UnityAssertEqualStringArray( const char** expected, void UnityAssertEqualStringArray(const char** expected,
const char** actual, const char** actual,
const UNITY_UINT32 num_elements, const UNITY_UINT32 num_elements,
const char* msg, const char* msg,
const UNITY_LINE_TYPE lineNumber) const UNITY_LINE_TYPE lineNumber)
{ {
UNITY_UINT32 i, j = 0; UNITY_UINT32 i, j = 0;
@@ -1061,12 +1057,12 @@ void UnityAssertEqualStringArray( const char** expected,
} }
/*-----------------------------------------------*/ /*-----------------------------------------------*/
void UnityAssertEqualMemory( UNITY_INTERNAL_PTR expected, void UnityAssertEqualMemory(UNITY_INTERNAL_PTR expected,
UNITY_INTERNAL_PTR actual, UNITY_INTERNAL_PTR actual,
const UNITY_UINT32 length, const UNITY_UINT32 length,
const UNITY_UINT32 num_elements, const UNITY_UINT32 num_elements,
const char* msg, const char* msg,
const UNITY_LINE_TYPE lineNumber) const UNITY_LINE_TYPE lineNumber)
{ {
UNITY_PTR_ATTRIBUTE const unsigned char* ptr_exp = (UNITY_PTR_ATTRIBUTE const unsigned char*)expected; UNITY_PTR_ATTRIBUTE const unsigned char* ptr_exp = (UNITY_PTR_ATTRIBUTE const unsigned char*)expected;
UNITY_PTR_ATTRIBUTE const unsigned char* ptr_act = (UNITY_PTR_ATTRIBUTE const unsigned char*)actual; UNITY_PTR_ATTRIBUTE const unsigned char* ptr_act = (UNITY_PTR_ATTRIBUTE const unsigned char*)actual;
@@ -1159,22 +1155,22 @@ void UnityIgnore(const char* msg, const UNITY_LINE_TYPE line)
UnityPrint(UnityStrIgnore); UnityPrint(UnityStrIgnore);
if (msg != NULL) if (msg != NULL)
{ {
UNITY_OUTPUT_CHAR(':'); UNITY_OUTPUT_CHAR(':');
UNITY_OUTPUT_CHAR(' '); UNITY_OUTPUT_CHAR(' ');
UnityPrint(msg); UnityPrint(msg);
} }
UNITY_IGNORE_AND_BAIL; UNITY_IGNORE_AND_BAIL;
} }
/*-----------------------------------------------*/ /*-----------------------------------------------*/
#if defined(UNITY_WEAK_ATTRIBUTE) #if defined(UNITY_WEAK_ATTRIBUTE)
UNITY_WEAK_ATTRIBUTE void setUp(void) { } UNITY_WEAK_ATTRIBUTE void setUp(void) { }
UNITY_WEAK_ATTRIBUTE void tearDown(void) { } UNITY_WEAK_ATTRIBUTE void tearDown(void) { }
#elif defined(UNITY_WEAK_PRAGMA) #elif defined(UNITY_WEAK_PRAGMA)
# pragma weak setUp #pragma weak setUp
void setUp(void) { } void setUp(void) { }
# pragma weak tearDown #pragma weak tearDown
void tearDown(void) { } void tearDown(void) { }
#endif #endif
/*-----------------------------------------------*/ /*-----------------------------------------------*/
void UnityDefaultTestRun(UnityTestFunction Func, const char* FuncName, const int FuncLineNum) void UnityDefaultTestRun(UnityTestFunction Func, const char* FuncName, const int FuncLineNum)
@@ -1248,7 +1244,7 @@ int UnityEnd(void)
char* UnityOptionIncludeNamed = NULL; char* UnityOptionIncludeNamed = NULL;
char* UnityOptionExcludeNamed = NULL; char* UnityOptionExcludeNamed = NULL;
int UnityVerbosity = 1; int UnityVerbosity = 1;
int UnityParseOptions(int argc, char** argv) int UnityParseOptions(int argc, char** argv)
{ {
@@ -1259,7 +1255,7 @@ int UnityParseOptions(int argc, char** argv)
{ {
if (argv[i][0] == '-') if (argv[i][0] == '-')
{ {
switch(argv[i][1]) switch (argv[i][1])
{ {
case 'l': /* list tests */ case 'l': /* list tests */
return -1; return -1;
@@ -1364,7 +1360,8 @@ int UnityStringArgumentMatches(const char* str)
/* look for the start of the next partial */ /* look for the start of the next partial */
ptr2 = ptr1; ptr2 = ptr1;
ptrf = 0; ptrf = 0;
do { do
{
ptr2++; ptr2++;
if ((ptr2[0] == ':') && (ptr2[1] != 0) && (ptr2[0] != '\'') && (ptr2[0] != '"') && (ptr2[0] != ',')) if ((ptr2[0] == ':') && (ptr2[1] != 0) && (ptr2[0] != '\'') && (ptr2[0] != '"') && (ptr2[0] != ','))
ptrf = &ptr2[1]; ptrf = &ptr2[1];