Skip to content

Commit b8aeaa2

Browse files
committed
writer-html: reword warnings about Coverity parsing
... to make them more self-explaining Resolves: https://issues.redhat.com/browse/OSH-422 Closes: #211
1 parent cbc6147 commit b8aeaa2

File tree

4 files changed

+11
-9
lines changed

4 files changed

+11
-9
lines changed

src/lib/writer-html.cc

+3-3
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ void writeParseWarnings(std::ostream &str, const TScanProps &props) {
147147
const int ratio = boost::lexical_cast<float>(itRatio->second);
148148
if (ratio < parsingRatioThr)
149149
str << "<p><b class='parseWarning'>warning:</b> "
150-
"low parsing ratio: " << ratio << "%</p>\n";
150+
"low ratio of compilation units successfully parsed by Coverity: " << ratio << "%</p>\n";
151151

152152
itCount = props.find("diffbase-cov-compilation-unit-count");
153153
itRatio = props.find("diffbase-cov-compilation-unit-ratio");
@@ -158,12 +158,12 @@ void writeParseWarnings(std::ostream &str, const TScanProps &props) {
158158
const int baseRatio = boost::lexical_cast<float>(itRatio->second);
159159
if (baseRatio < parsingRatioThr && baseRatio < ratio)
160160
str << "<p><b class='parseWarning'>warning:</b> "
161-
"low parsing ratio in diff base: "
161+
"low ratio of compilation units successfully parsed by Coverity in diff base: "
162162
<< baseRatio << "%</p>\n";
163163

164164
if (!count || 100 * baseCount / count < parsingOldToNewRatioThr)
165165
str << "<p><b class='parseWarning'>warning:</b> "
166-
"low count of parsed units in diff base: "
166+
"low count of compilation units successfully parsed by Coverity in diff base: "
167167
<< baseCount << "</p>\n";
168168
}
169169
catch (boost::bad_lexical_cast &) {

tests/cshtml/0001-smoke/old/scan-results.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"analyzer-version-cppcheck": "1.80",
66
"analyzer-version-gcc": "4.8.5",
77
"analyzer-version-shellcheck": "0.4.7",
8-
"cov-compilation-unit-count": 149,
8+
"cov-compilation-unit-count": 49,
99
"cov-compilation-unit-ratio": 100,
1010
"cov-lines-processed": 127521,
1111
"cov-time-elapsed-analysis": "00:01:51",

tests/cshtml/0001-smoke/scan-results.html

+5-3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
<body>
2222
<h1>curl-7.60.0-1.el8+7</h1>
2323
<a href='scan-results.err'>[Show plain-text results]</a>
24+
<p><b class='parseWarning'>warning:</b> low ratio of compilation units successfully parsed by Coverity: 80%</p>
25+
<p><b class='parseWarning'>warning:</b> low count of compilation units successfully parsed by Coverity in diff base: 49</p>
2426
<h2>List of Findings</h2>
2527
<pre>
2628
<a id='def1'></a><b>Error: <span class='checker'>SHELLCHECK_WARNING</span>:</b> <a href='#def1'>[#def1]</a>
@@ -1430,11 +1432,11 @@ <h2>Scan Properties</h2>
14301432
<tr class='oddRow'><td class='scanPropName'>analyzer-version-cppcheck</td><td>1.80</td></tr>
14311433
<tr><td class='scanPropName'>analyzer-version-gcc</td><td>8.1.1</td></tr>
14321434
<tr class='oddRow'><td class='scanPropName'>analyzer-version-shellcheck</td><td>0.4.7</td></tr>
1433-
<tr><td class='scanPropName'>cov-compilation-unit-count</td><td>193</td></tr>
1434-
<tr class='oddRow'><td class='scanPropName'>cov-compilation-unit-ratio</td><td>100</td></tr>
1435+
<tr><td class='scanPropName'>cov-compilation-unit-count</td><td>154</td></tr>
1436+
<tr class='oddRow'><td class='scanPropName'>cov-compilation-unit-ratio</td><td>80</td></tr>
14351437
<tr><td class='scanPropName'>cov-lines-processed</td><td>174359</td></tr>
14361438
<tr class='oddRow'><td class='scanPropName'>cov-time-elapsed-analysis</td><td>00:01:55</td></tr>
1437-
<tr><td class='scanPropName'>diffbase-cov-compilation-unit-count</td><td>149</td></tr>
1439+
<tr><td class='scanPropName'>diffbase-cov-compilation-unit-count</td><td>49</td></tr>
14381440
<tr class='oddRow'><td class='scanPropName'>diffbase-cov-compilation-unit-ratio</td><td>100</td></tr>
14391441
<tr><td class='scanPropName'>exit-code</td><td>0</td></tr>
14401442
<tr class='oddRow'><td class='scanPropName'>host</td><td>cov01.lab.eng.brq.redhat.com</td></tr>

tests/cshtml/0001-smoke/scan-results.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
"analyzer-version-cppcheck": "1.80",
66
"analyzer-version-gcc": "8.1.1",
77
"analyzer-version-shellcheck": "0.4.7",
8-
"cov-compilation-unit-count": 193,
9-
"cov-compilation-unit-ratio": 100,
8+
"cov-compilation-unit-count": 154,
9+
"cov-compilation-unit-ratio": 80,
1010
"cov-lines-processed": 174359,
1111
"cov-time-elapsed-analysis": "00:01:55",
1212
"exit-code": 0,

0 commit comments

Comments
 (0)