-
Notifications
You must be signed in to change notification settings - Fork 3
/
spotbugs-exclude.xml
84 lines (70 loc) · 3.28 KB
/
spotbugs-exclude.xml
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
<?xml version="1.0" encoding="UTF-8"?>
<FindBugsFilter
xmlns="https://github.com/spotbugs/filter/4.8.4"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="https://github.com/spotbugs/filter/4.8.4 https://raw.githubusercontent.com/spotbugs/spotbugs/4.8.4/spotbugs/etc/findbugsfilter.xsd">
<Match>
<Bug category="MALICIOUS_CODE"/>
</Match>
<!-- Ignore activators -->
<Match>
<Or>
<Class name="~.*Activator" />
<Class name="~.*LibPlugin" />
</Or>
</Match>
<!-- Ignore generated java classes -->
<Match>
<Or>
<Source name="~.*src-gen.*.java" />
<Source name="~.*xtend-gen.*.java" />
<Source name="~.*src-dmf.*.java" />
</Or>
</Match>
<Match>
<Or>
<!-- Ignore this warning since the variable might be accessed in a downstream
project -->
<Bug pattern="URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD" />
<!-- Ignore this warning since we allow equality between objects that
are not consistent with their type hierarchy. An example for this is a bean
category assignment and a category assignment -->
<Bug pattern="EQ_CHECK_FOR_OPERAND_NOT_COMPATIBLE_WITH_THIS" />
<!-- Ignore these warnings since it is not clear whether it is an actual
issue or of spotbugs is giving us false positives. Should be rechecked in
the future -->
<Bug pattern="SE_BAD_FIELD" />
<Bug pattern="SE_BAD_FIELD_STORE" />
<!-- Temporarily ignore this warning since addressing it in some cases
seems to involve some more refactoring. Should be rechecked in the future -->
<Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE" />
<!-- Ignore this warning since we are using instanceof in test cases to
assert the correct type of some objects. It should be rechecked in the future
if it is useful to allow this rule for any Non-Test class. -->
<Bug pattern="BC_VACUOUS_INSTANCEOF" />
<!-- Ignore this warning since it is not clear if it is an actual issúe
or a false positive of spotbugs. Should be rechecked in the future -->
<Bug pattern="EQ_DOESNT_OVERRIDE_EQUALS" />
<!-- Ignore this warning since it seems to be giving false positives.
The spotbugs documentation also states that this warning might be bugged
in some cases. -->
<Bug pattern="SF_SWITCH_NO_DEFAULT" />
<!-- Ignore this warning since it is not clear from the documentation
what the issue is. Should be rechecked in the future. -->
<Bug pattern="UC_USELESS_OBJECT" />
<!-- Ignore this warning since there are a few static assignments in the code
where it is not clear if they are good or bad practice or necessary.
Should be rechecked in the future. -->
<Bug pattern="ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD" />
<!-- Ignore this warning, since it is not clear if it is a false positive or not.
Should be rechecked in the future. -->
<Bug pattern="MS_PKGPROTECT" />
<Bug pattern="MS_CANNOT_BE_FINAL" />
<!-- Ignore this warning, since there is a case where it makes sense to have
switch fallthrough. -->
<Bug pattern="SF_SWITCH_FALLTHROUGH" />
<!-- Suggests a modification to a pontetially deprecated API. -->
<Bug pattern="DP_CREATE_CLASSLOADER_INSIDE_DO_PRIVILEGED" />
</Or>
</Match>
</FindBugsFilter>