TestPlan: Do not re-declare bool result

Fixing this error makes TestPlan pass again.

Signed-off-by: Rick Walsh <rickmwalsh@gmail.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Rick Walsh 2015-08-30 15:40:34 +10:00 committed by Dirk Hohndel
parent d22a135343
commit d15a1db428

View file

@ -204,7 +204,7 @@ bool compareDecoTime(int actualRunTimeSeconds, int benchmarkRunTimeSeconds, int
// If the calculated run time equals the expected run time, do a simple comparison // If the calculated run time equals the expected run time, do a simple comparison
if (actualRunTimeSeconds == benchmarkRunTimeSeconds) { if (actualRunTimeSeconds == benchmarkRunTimeSeconds) {
bool result = true; result = true;
} else { } else {
/* We want the difference between the expected and calculated total run time to be not more than /* We want the difference between the expected and calculated total run time to be not more than
* 1% of total run time + 1 minute */ * 1% of total run time + 1 minute */
@ -219,7 +219,7 @@ bool compareDecoTime(int actualRunTimeSeconds, int benchmarkRunTimeSeconds, int
permilDifferenceAllowed * 0.1, absoluteDifferenceAllowedSeconds, totalDifferenceAllowed); permilDifferenceAllowed * 0.1, absoluteDifferenceAllowedSeconds, totalDifferenceAllowed);
qInfo("total difference = %d seconds", totalDifference); qInfo("total difference = %d seconds", totalDifference);
bool result = (totalDifference <= totalDifferenceAllowed); result = (totalDifference <= totalDifferenceAllowed);
} }
if ((knownSsrfRunTimeSeconds > 0) && (actualRunTimeSeconds != knownSsrfRunTimeSeconds)) { if ((knownSsrfRunTimeSeconds > 0) && (actualRunTimeSeconds != knownSsrfRunTimeSeconds)) {
QWARN("Calculated run time does not match known Subsurface runtime"); QWARN("Calculated run time does not match known Subsurface runtime");