Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(286)

Side by Side Diff: build/android/findbugs_filter/findbugs_exclude.xml

Issue 1000793002: [Android] Incorporate findbugs into android builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address cjhopman's comment + rebase Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « build/android/findbugs_diff.py ('k') | build/android/pylib/utils/findbugs.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <?xml version="1.0" encoding="UTF-8"?> 1 <?xml version="1.0" encoding="UTF-8"?>
2 <!-- 2 <!--
3 Copyright (c) 2012 The Chromium Authors. All rights reserved. 3 Copyright (c) 2012 The Chromium Authors. All rights reserved.
4 Use of this source code is governed by a BSD-style license that can be 4 Use of this source code is governed by a BSD-style license that can be
5 found in the LICENSE file. 5 found in the LICENSE file.
6 --> 6 -->
7 7
8 <!-- 8 <!--
9 Documentation: http://findbugs.sourceforge.net/manual/filter.html 9 Documentation: http://findbugs.sourceforge.net/manual/filter.html
10 In particular, ~ at the start of a string means it's a regex. 10 In particular, ~ at the start of a string means it's a regex.
11 --> 11 -->
12 <FindBugsFilter> 12 <FindBugsFilter>
13 <!-- Skip the generated resource classes (including nested classes). --> 13 <!-- Skip the generated resource classes (including nested classes). -->
14 <Match> 14 <Match>
15 <Class name="~org\.chromium\..*\.R(\$\w+)?" /> 15 <Class name="~.*\.R(\$\w+)?" />
16 </Match> 16 </Match>
17 <Match> 17 <Match>
18 <Class name="~org\.chromium\..*\.Manifest(\$\w+)?" /> 18 <Class name="~org\.chromium\..*\.Manifest(\$\w+)?" />
19 </Match> 19 </Match>
20 <Bug pattern="DM_STRING_CTOR" />
20 <!-- Ignore "reliance on default String encoding" warnings, as we're not multi -platform --> 21 <!-- Ignore "reliance on default String encoding" warnings, as we're not multi -platform -->
21 <Match>
22 <Class name="~org\.chromium\.chrome\.browser\.ChromiumApplication.*" />
23 <Method name="onCancel" />
24 <Bug code="Dm" />
25 </Match>
26 <Match>
27 <Class name="~org\.chromium\.chrome\.browser\.ChromiumApplication.*" />
28 <Method name="onClick" />
29 <Bug code="Dm" />
30 </Match>
31 <Bug pattern="DM_DEFAULT_ENCODING" /> 22 <Bug pattern="DM_DEFAULT_ENCODING" />
32 <!-- Ignore bugs that are often false-positives in test code -->
33 <Match>
34 <Class name="~org\.chromium\..*Test(\$\w+)?" />
35 <Or>
36 <Bug code="DLS,UrF" />
37 <Bug pattern="DM_GC" />
38 </Or>
39 </Match>
40 <!--
41 crbug.com/449101
42 Ignore findbugs plugin test cases.
43 -->
44 <Match>
45 <Or>
46 <Class name="~org\.chromium\.tools\.findbugs\.plugin\.SimpleSynchronizedMe thod" />
47 <Class name="~org\.chromium\.tools\.findbugs\.plugin\.SimpleSynchronizedSt aticMethod" />
48 <Class name="~org\.chromium\.tools\.findbugs\.plugin\.SimpleSynchronizedTh is" />
49 </Or>
50 </Match>
51 </FindBugsFilter> 23 </FindBugsFilter>
OLDNEW
« no previous file with comments | « build/android/findbugs_diff.py ('k') | build/android/pylib/utils/findbugs.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698