espressif/Unity
1
0
Fork 0
mirror of https://github.com/ThrowTheSwitch/Unity.git synced 2025-06-25 18:57:58 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix unreachable code in Unity on fully covered switch statements

Browse Source 
The clang compiler warns here with -Wunreachable-code
 The enum's switch statement covers all cases, so default is unused
 Leave the break in the code as a comment, to be more clear
 The history is this default case was added in f6bb7162 - compiler warning.
 Then the break was added in c6dc96f3.
This commit is contained in:
jsalling
2016-02-27 10:26:04 -06:00
parent e84a7c0a7d
commit 7a0875f4f3
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/unity.c
View File

@ -775,7 +775,7 @@ void UnityAssertFloatSpecial(const _UF actual,
break; break;
default: default:
break; /*break // unreachable code */;
} }
if (is_trait != should_be_trait) if (is_trait != should_be_trait)
@ -938,7 +938,7 @@ void UnityAssertDoubleSpecial(const _UD actual,
break; break;
default: default:
break; /*break // unreachable code*/;
} }
if (is_trait != should_be_trait) if (is_trait != should_be_trait)