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

Side by Side Diff: tools/android/checkstyle/chromium-style-5.0.xml

Issue 517403002: [Android checkstyle] Allow underscores in package names. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: warning -> error Created 6 years, 3 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 | « no previous file | no next file » | 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 <!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.3//EN" "http: //www.puppycrawl.com/dtds/configuration_1_3.dtd"> 2 <!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.3//EN" "http: //www.puppycrawl.com/dtds/configuration_1_3.dtd">
3 3
4 <!-- 4 <!--
5 See installation instructions: https://sites.google.com/a/chromium.org/dev/che ckstyle 5 See installation instructions: https://sites.google.com/a/chromium.org/dev/che ckstyle
6 --> 6 -->
7 <module name="Checker"> 7 <module name="Checker">
8 <property name="severity" value="warning"/> 8 <property name="severity" value="warning"/>
9 <property name="charset" value="UTF-8"/> 9 <property name="charset" value="UTF-8"/>
10 <module name="TreeWalker"> 10 <module name="TreeWalker">
(...skipping 20 matching lines...) Expand all
31 </module> 31 </module>
32 <module name="JavadocMethod"> 32 <module name="JavadocMethod">
33 <property name="severity" value="warning"/> 33 <property name="severity" value="warning"/>
34 <property name="scope" value="public"/> 34 <property name="scope" value="public"/>
35 <property name="allowMissingParamTags" value="true"/> 35 <property name="allowMissingParamTags" value="true"/>
36 <property name="allowMissingPropertyJavadoc" value="true"/> 36 <property name="allowMissingPropertyJavadoc" value="true"/>
37 <property name="allowMissingReturnTag" value="true"/> 37 <property name="allowMissingReturnTag" value="true"/>
38 <property name="allowMissingThrowsTags" value="true"/> 38 <property name="allowMissingThrowsTags" value="true"/>
39 </module> 39 </module>
40 <module name="PackageName"> 40 <module name="PackageName">
41 <property name="severity" value="warning"/> 41 <property name="severity" value="error"/>
42 <property name="format" value="^[a-z]+(\.[a-z][a-z0-9]{1,})*$"/> 42 <property name="format" value="^[a-z]+(\.[a-z][a-z0-9_]{1,})*$"/>
43 </module> 43 </module>
44 <module name="TypeName"> 44 <module name="TypeName">
45 <property name="severity" value="warning"/> 45 <property name="severity" value="warning"/>
46 </module> 46 </module>
47 <module name="ConstantName"> 47 <module name="ConstantName">
48 <property name="severity" value="error"/> 48 <property name="severity" value="error"/>
49 <property name="format" value="^([A-Z][A-Z0-9]*(_[A-Z0-9]+)*)|(s[A-Z][a-zA -Z0-9]*)$"/> 49 <property name="format" value="^([A-Z][A-Z0-9]*(_[A-Z0-9]+)*)|(s[A-Z][a-zA -Z0-9]*)$"/>
50 <message key="name.invalidPattern" value="Static final field names must ei ther be all caps (e.g. int HEIGHT_PX) for 'true' constants, or start with s (e.g . AtomicInteger sNextId or Runnable sSuspendTask) for fields with mutable state or that don't 'feel' like constants."/> 50 <message key="name.invalidPattern" value="Static final field names must ei ther be all caps (e.g. int HEIGHT_PX) for 'true' constants, or start with s (e.g . AtomicInteger sNextId or Runnable sSuspendTask) for fields with mutable state or that don't 'feel' like constants."/>
51 </module> 51 </module>
52 <!-- Non-public, non-static field names start with m. --> 52 <!-- Non-public, non-static field names start with m. -->
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 <module name="RegexpSingleline"> 151 <module name="RegexpSingleline">
152 <property name="severity" value="error"/> 152 <property name="severity" value="error"/>
153 <property name="format" value="[ \t]+$"/> 153 <property name="format" value="[ \t]+$"/>
154 <property name="message" value="Trailing whitespace"/> 154 <property name="message" value="Trailing whitespace"/>
155 </module> 155 </module>
156 <module name="RegexpHeader"> 156 <module name="RegexpHeader">
157 <property name="severity" value="error"/> 157 <property name="severity" value="error"/>
158 <property name="header" value="^// Copyright 20\d\d The Chromium Authors. Al l rights reserved.$\n^// Use of this source code is governed by a BSD-style lice nse that can be$\n^// found in the LICENSE file.$"/> 158 <property name="header" value="^// Copyright 20\d\d The Chromium Authors. Al l rights reserved.$\n^// Use of this source code is governed by a BSD-style lice nse that can be$\n^// found in the LICENSE file.$"/>
159 </module> 159 </module>
160 </module> 160 </module>
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698