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

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

Issue 455523002: [Checkstyle] Update static variable and constant names to match style guides. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: yfriendman's nits Created 6 years, 4 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 | « remoting/android/java/src/org/chromium/chromoting/HelpActivity.java ('k') | 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 27 matching lines...) Expand all
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="warning"/>
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="warning"/> 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. -->
53 <module name="MemberName"> 53 <module name="MemberName">
54 <property name="severity" value="error"/> 54 <property name="severity" value="error"/>
55 <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/> 55 <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/>
56 <property name="applyToPublic" value="false"/> 56 <property name="applyToPublic" value="false"/>
57 <message key="name.invalidPattern" value="Non-public, non-static field n ames start with m."/> 57 <message key="name.invalidPattern" value="Non-public, non-static field n ames start with m."/>
58 </module> 58 </module>
59 <!-- Static field names start with s. --> 59 <!-- Static field names start with s. -->
60 <module name="StaticVariableName"> 60 <module name="StaticVariableName">
61 <property name="severity" value="error"/>
61 <property name="format" value="^s[A-Z][a-zA-Z0-9]*$"/> 62 <property name="format" value="^s[A-Z][a-zA-Z0-9]*$"/>
62 <property name="applyToPublic" value="false"/> 63 <property name="applyToPublic" value="false"/>
63 <message key="name.invalidPattern" value="Static field names start with s."/> 64 <message key="name.invalidPattern" value="Static field names start with s."/>
64 </module> 65 </module>
65 <module name="MethodName" /> 66 <module name="MethodName" />
66 <module name="ParameterName"> 67 <module name="ParameterName">
67 <property name="severity" value="warning"/> 68 <property name="severity" value="warning"/>
68 </module> 69 </module>
69 <module name="LocalFinalVariableName"> 70 <module name="LocalFinalVariableName">
70 <property name="severity" value="warning"/> 71 <property name="severity" value="warning"/>
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 <module name="RegexpSingleline"> 151 <module name="RegexpSingleline">
151 <property name="severity" value="error"/> 152 <property name="severity" value="error"/>
152 <property name="format" value="[ \t]+$"/> 153 <property name="format" value="[ \t]+$"/>
153 <property name="message" value="Trailing whitespace"/> 154 <property name="message" value="Trailing whitespace"/>
154 </module> 155 </module>
155 <module name="RegexpHeader"> 156 <module name="RegexpHeader">
156 <property name="severity" value="error"/> 157 <property name="severity" value="error"/>
157 <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.$"/>
158 </module> 159 </module>
159 </module> 160 </module>
OLDNEW
« no previous file with comments | « remoting/android/java/src/org/chromium/chromoting/HelpActivity.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698