blob: 2f6dd49b51651113fa475a2de45bdeb1bbc49122 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
|
// File to suppress cppcheck warnings for files that will not be fixed.
// Does not suppress warnings where an additional occurrence of the warning may be of interest.
// Configured for cppcheck 2.20
// Just a report of how many checkers are run
checkersReport
// This just warns that cppcheck isn't exhaustive and it still appears in exhaustive mode
normalCheckLevelMaxBranches
// Coding style is to use assignments in constructor when there are many
// members to initialize or the initialization is complex or has comments.
useInitializationList
// These may be interesting but its not clear without examining each instance closely
// Would have to ensure that any_of/all_of has same early/late exits as current code and
// produces same result on empty collections
useStlAlgorithm
// Written with variable for consistency
knownArgument:scintilla/src/SparseVector.h
// cppcheck 2.11 can't find system headers on Win32.
missingIncludeSystem
// The cast converts from 'unsigned char ' to 'char' so isn't unused.
// Redundant code: Found unused cast of expression 'leadByte'
constStatement:scintilla/src/Document.cxx
// Some non-explicit constructors are used for conversions or are private to lexers
noExplicitConstructor
// RangesCopy is deliberately returning a copy.
// The copy is always mutated so returning a refererence just enables lifetime problems.
returnByReference:scintilla/src/Selection.h
// MarginView access to all bits is safe and is better defined in later versions of C++
shiftTooManyBitsSigned:scintilla/src/MarginView.cxx
// Doesn't understand changing dropWentOutside in Editor
knownConditionTrueFalse:scintilla/win32/ScintillaWin.cxx
// GetData is implementing interface so shouldn't add const
constParameterPointer:scintilla/win32/ScintillaWin.cxx
// Doesn't handle intptr_t (long long) being signed
knownConditionTrueFalse:scintilla/src/Editor.cxx
knownConditionTrueFalse:scintilla/src/EditView.cxx
// This makes no sense
CastIntegerToAddressAtReturn:scintilla\src\DBCS.cxx
// Preprocessor #if with *_CHECK_VERSION(...) *_VERSION_CHECK(...)
syntaxError:scintilla/gtk/Wrappers.h
syntaxError:scintilla/gtk/PlatGTK.cxx
syntaxError:scintilla/gtk/ScintillaGTKAccessible.cxx
syntaxError:scintilla/qt/ScintillaEditBase/PlatQt.cpp
syntaxError:scintilla/qt/ScintillaEditBase/ScintillaEditBase.cpp
syntaxError:scintilla/qt/ScintillaEditBase/ScintillaQt.cpp
// G_DEFINE_TYPE is too complex to pass to cppcheck
unknownMacro:scintilla/gtk/PlatGTK.cxx
// G_END_DECLS
unknownMacro:scintilla/gtk/scintilla-marshal.h
// Marshaling hidden from cppcheck
unusedFunction:scintilla/gtk/scintilla-marshal.c
// Difficult to test accessibility so don't change
constParameterPointer:scintilla/gtk/ScintillaGTKAccessible.cxx
constVariableReference:scintilla/gtk/ScintillaGTKAccessible.cxx
constVariablePointer:scintilla/gtk/ScintillaGTKAccessible.cxx
// moc_ files show #error as they are not built with standard context
preprocessorErrorDirective:scintilla/qt/**.cpp
// Doesn't understand Qt slots macro
unknownMacro:scintilla/qt/ScintillaEditBase/*.h
// Provided for other projects
unusedFunction:scintilla/qt/ScintillaEdit/ScintillaDocument.cpp
// Provided for other projects
unusedFunction:scintilla/call/ScintillaCall.cxx
// Don't want to unnecessarily allow const pointers to be made mutable through reinterpret_cast
constParameterPointer:scintilla/call/ScintillaCall.cxx
constParameterPointer:scintilla/src/Editor.cxx
constVariablePointer:scintilla/src/Editor.cxx
// The performance cost of by-value passing is often small and using a reference decreases
// code legibility.
passedByValue
// For conciseness, the testing code is much sloppier than the library
// Suppress everything in catch.hpp as won't be changing
*:scintilla/test/unit/catch.hpp
// unread variables and overwritten without read in tests
unreadVariable:scintilla/test/unit/*.cxx
// Checks for moves move to variables that are not read but the moved from is checked
accessMoved:scintilla/test/unit/*.cxx
// 'obviously' true tests are still useful
knownConditionTrueFalse:scintilla/test/unit/*.cxx
// fields in TEST_CASE could be restricted to SECTION but not interesting
variableScope:scintilla/test/unit/*.cxx
// overwritten without read in tests
redundantInitialization:scintilla/test/unit/*.cxx
redundantAssignment:scintilla/test/unit/*.cxx
unusedVariable:scintilla/test/unit/*.cxx
// doesn't understand cppcheck TEST_CASE
syntaxError:scintilla/test/unit/*.cxx
// doesn't understand cppcheck generated functions
unusedFunction:scintilla/test/unit/*.cxx
// cppcheck fails REQUIRE from Catch
comparisonOfFuncReturningBoolError:scintilla/test/unit/*.cxx
|