Skip to content

Commit ea95cc6

Browse files
committed
csfilter: abstract out precise numbers for IDENTIFIER_TYPO
1 parent 1760607 commit ea95cc6

File tree

8 files changed

+34
-0
lines changed

8 files changed

+34
-0
lines changed

csfilter.cc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,9 @@ struct MsgFilter::Private {
110110

111111
// artificial field names of anonymous unions that Coverity produces
112112
addMsgFilter("", "__C[0-9]+");
113+
114+
// used by IDENTIFIER_TYPO (but applies generally)
115+
addMsgFilter("", "at least [0-9][0-9]* times.$");
113116
}
114117
};
115118

tests/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,3 +175,4 @@ test_csdiff(diff-misc 02)
175175
test_csdiff(diff-misc 03)
176176
test_csdiff(diff-misc 04)
177177
test_csdiff(diff-misc 05)
178+
test_csdiff(diff-misc 06)

tests/diff-misc/06-add-z.err

Whitespace-only changes.

tests/diff-misc/06-add.err

Whitespace-only changes.

tests/diff-misc/06-fix-z.err

Whitespace-only changes.

tests/diff-misc/06-fix.err

Whitespace-only changes.

tests/diff-misc/06-new.err

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
Error: IDENTIFIER_TYPO:
2+
anaconda-21.48.22.90/pyanaconda/ui/gui/spokes/source.py:1388: identifier_typo: Using "mirorlist" appears to be a typo:
3+
* Identifier "mirorlist" is only known to be referenced here, or in copies of this code.
4+
* Identifier "mirrorlist" is referenced elsewhere at least 27 times.
5+
anaconda-21.48.22.90/pyanaconda/packaging/__init__.py:1046: identifier_use: Example 1: Using identifier "mirrorlist".
6+
anaconda-21.48.22.90/pyanaconda/packaging/yumpayload.py:732: identifier_use: Example 2: Using identifier "mirrorlist".
7+
anaconda-21.48.22.90/pyanaconda/packaging/yumpayload.py:879: identifier_use: Example 3: Using identifier "mirrorlist".
8+
anaconda-21.48.22.90/pyanaconda/packaging/yumpayload.py:726: identifier_use: Example 4: Using identifier "mirrorlist".
9+
anaconda-21.48.22.90/pyanaconda/packaging/yumpayload.py:335: identifier_use: Example 5: Using identifier "mirrorlist" (2 total uses in this function).
10+
anaconda-21.48.22.90/pyanaconda/ui/gui/spokes/source.py:1388: remediation: Should identifier "mirorlist" be replaced by "mirrorlist"?
11+
# 1386| url = self._repoUrlEntry.get_text().strip()
12+
# 1387| if self._repoMirrorlistCheckbox.get_active():
13+
# 1388|-> repo.mirorlist = proto + url
14+
# 1389| else:
15+
# 1390| repo.baseurl = proto + url

tests/diff-misc/06-old.err

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
Error: IDENTIFIER_TYPO:
2+
anaconda-21.48.22.56/pyanaconda/ui/gui/spokes/source.py:1345: identifier_typo: Using "mirorlist" appears to be a typo:
3+
* Identifier "mirorlist" is only known to be referenced here, or in copies of this code.
4+
* Identifier "mirrorlist" is referenced elsewhere at least 26 times.
5+
anaconda-21.48.22.56/pyanaconda/packaging/__init__.py:956: identifier_use: Example 1: Using identifier "mirrorlist".
6+
anaconda-21.48.22.56/pyanaconda/packaging/yumpayload.py:719: identifier_use: Example 2: Using identifier "mirrorlist".
7+
anaconda-21.48.22.56/pyanaconda/packaging/yumpayload.py:866: identifier_use: Example 3: Using identifier "mirrorlist".
8+
anaconda-21.48.22.56/pyanaconda/packaging/yumpayload.py:713: identifier_use: Example 4: Using identifier "mirrorlist".
9+
anaconda-21.48.22.56/pyanaconda/packaging/yumpayload.py:334: identifier_use: Example 5: Using identifier "mirrorlist" (2 total uses in this function).
10+
anaconda-21.48.22.56/pyanaconda/ui/gui/spokes/source.py:1345: remediation: Should identifier "mirorlist" be replaced by "mirrorlist"?
11+
# 1343| url = self._repoUrlEntry.get_text().strip()
12+
# 1344| if self._repoMirrorlistCheckbox.get_active():
13+
# 1345|-> repo.mirorlist = proto + url
14+
# 1346| else:
15+
# 1347| repo.baseurl = proto + url

0 commit comments

Comments
 (0)