From 782eba3a0c74009e24708d28fd520b1ba919d326 Mon Sep 17 00:00:00 2001 From: Zack Galbreath Date: Thu, 22 Aug 2013 15:20:30 -0400 Subject: [PATCH] CTest: Fix GTM coverage parsing line offset bug In cmParseGTMCoverage::ReadMCovFile, initialize the lineoffset variable. Also set lastoffset only if the function is found (thanks to Bill Hoffman). --- Source/CTest/cmParseGTMCoverage.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Source/CTest/cmParseGTMCoverage.cxx b/Source/CTest/cmParseGTMCoverage.cxx index 5dfcfe50a..6b4adb4bd 100644 --- a/Source/CTest/cmParseGTMCoverage.cxx +++ b/Source/CTest/cmParseGTMCoverage.cxx @@ -98,7 +98,7 @@ bool cmParseGTMCoverage::ReadMCovFile(const char* file) bool found = this->FindMumpsFile(routine, filepath); if(found) { - int lineoffset; + int lineoffset = 0; if(this->FindFunctionInMumpsFile(filepath, function, lineoffset)) @@ -106,8 +106,8 @@ bool cmParseGTMCoverage::ReadMCovFile(const char* file) cmCTestCoverageHandlerContainer::SingleFileCoverageVector& coverageVector = this->Coverage.TotalCoverage[filepath]; coverageVector[lineoffset + linenumber] += count; + lastoffset = lineoffset; } - lastoffset = lineoffset; } else {