Diff: TestManager.cpp

Revision:

26:9eefab9e28df

Parent:

24:8f0f9551122a

Child:

27:0297dbc3252b

--- a/TestManager.cpp	Thu Sep 06 13:35:25 2012 +0000
+++ b/TestManager.cpp	Thu Sep 06 14:28:01 2012 +0000
@@ -28,7 +28,7 @@
 }
 
 int TestManager::getTestProfileLength(TestProfile profile) {
-    if(profile>=0 && profile <TESTS_END) {
+    if(profile < TESTS_END) { // no test for >= 0 since TestProfile is unsigned
       return gTestProfileLengths[profile];
    } else {
       LOG("Test profile out of bounds. Must be between 0 and %d",TESTS_END-1);
@@ -38,7 +38,7 @@
 
 int TestManager::executeTestProfile(TestProfile profile) {
    int numPassed = 0;
-   if(profile>=0 && profile <TESTS_END) {
+   if(profile <TESTS_END) { // no test for >= 0 since TestProfile is unsigned
       numPassed = executeTestList(gTestProfiles[profile], gTestProfileLengths[profile]);
    } else {
       LOG("Test profile out of bounds. Must be between 0 and %d",TESTS_END-1);