cppcheck.rules 2.52 KB
Newer Older
Paul's avatar
Paul committed
1
<?xml version="1.0"?>
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<rule>
    <pattern> [;{}] [*] \w+? (\+\+|\-\-) ; </pattern>
    <message>
        <id>UnusedDeref</id>
        <severity>style</severity>
        <summary>Redundant * found, "*p++" is the same as "*(p++)".</summary>
    </message>
</rule>
<rule>
    <pattern> if \( ([!] )*?(strlen) \( \w+? \) ([>] [0] )*?\) { </pattern>
    <message>
        <id>StrlenEmptyString</id>
        <severity>performance</severity>
        <summary>Using strlen() to check if a string is empty is not efficient.</summary>
    </message>
</rule>
<rule>
    <pattern> [;{}] [*] \w+? (\+\+|\-\-) ; </pattern>
    <message>
        <id>UnusedDeref</id>
        <severity>style</severity>
        <summary>Redundant * found, "*p++" is the same as "*(p++)".</summary>
    </message>
</rule>
Paul's avatar
Paul committed
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
<rule>
    <tokenlist>define</tokenlist>
    <pattern>define [0-9A-Z_^a-z]*[a-z]</pattern>
    <message>
        <id>defineUpperCase</id>
        <severity>style</severity>
        <summary>Macros must be uppercase</summary>
    </message>
</rule>
<rule>
    <tokenlist>define</tokenlist>
    <pattern>define (MIGRAP|[^H]{6})[^H][^_]</pattern>
    <message>
        <id>definePrefix</id>
        <severity>style</severity>
        <summary>Macros must be prefixed with MIGRAPH_</summary>
    </message>
</rule>
<rule>
Paul's avatar
Paul committed
45
    <pattern>(memcpy|strcpy|strncpy|strcat|strncat) \(</pattern>
Paul's avatar
Paul committed
46
47
48
    <message>
        <id>useStlAlgorithms</id>
        <severity>style</severity>
Paul's avatar
Paul committed
49
        <summary>Use std::copy instead</summary>
Paul's avatar
Paul committed
50
51
52
53
54
55
56
    </message>
</rule>
<rule>
    <pattern>memset \(</pattern>
    <message>
        <id>useStlAlgorithms</id>
        <severity>style</severity>
Paul's avatar
Paul committed
57
58
59
60
61
62
63
64
        <summary>Use std::fill instead</summary>
    </message>
</rule>
<rule>
    <pattern>memcmp \(</pattern>
    <message>
        <id>useStlAlgorithms</id>
        <severity>style</severity>
65
        <summary>Use std::equal_range instead</summary>
Paul's avatar
Paul committed
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
    </message>
</rule>
<rule>
    <pattern>memchr \(</pattern>
    <message>
        <id>useStlAlgorithms</id>
        <severity>style</severity>
        <summary>Use std::find instead</summary>
    </message>
</rule>
<rule>
    <pattern>(fclose|free|hipFree) \(</pattern>
    <message>
        <id>useManagePointer</id>
        <severity>style</severity>
        <summary>Use manage pointer for resource management</summary>
    </message>
</rule>
<rule>
    <tokenlist>normal</tokenlist>
    <pattern>! !</pattern>
    <message>
        <id>doubleNegative</id>
        <severity>style</severity>
        <summary>Double negative is always positive</summary>
Paul's avatar
Paul committed
91
92
    </message>
</rule>