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

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

Issue 452613002: [Checkstyle] Update class member variable names to match style guides. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: boliu 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
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 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
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="warning"/>
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="format" value="^m[A-Z][a-zA-Z0-9]*$"/> 55 <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/>
55 <property name="applyToPublic" value="false"/> 56 <property name="applyToPublic" value="false"/>
56 <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."/>
57 </module> 58 </module>
58 <!-- Static field names start with s. --> 59 <!-- Static field names start with s. -->
59 <module name="StaticVariableName"> 60 <module name="StaticVariableName">
60 <property name="format" value="^s[A-Z][a-zA-Z0-9]*$"/> 61 <property name="format" value="^s[A-Z][a-zA-Z0-9]*$"/>
61 <property name="applyToPublic" value="false"/> 62 <property name="applyToPublic" value="false"/>
62 <message key="name.invalidPattern" value="Static field names start with s."/> 63 <message key="name.invalidPattern" value="Static field names start with s."/>
63 </module> 64 </module>
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 <module name="RegexpSingleline"> 150 <module name="RegexpSingleline">
150 <property name="severity" value="error"/> 151 <property name="severity" value="error"/>
151 <property name="format" value="[ \t]+$"/> 152 <property name="format" value="[ \t]+$"/>
152 <property name="message" value="Trailing whitespace"/> 153 <property name="message" value="Trailing whitespace"/>
153 </module> 154 </module>
154 <module name="RegexpHeader"> 155 <module name="RegexpHeader">
155 <property name="severity" value="error"/> 156 <property name="severity" value="error"/>
156 <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.$"/> 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.$"/>
157 </module> 158 </module>
158 </module> 159 </module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698