commit 540b072483129601b7a856bc0a3a44e3c325586d Author: iwakeh iwakeh@torproject.org Date: Tue Jan 3 18:00:00 2017 +0100

Limit line length to 80. Removed property 'maxLineLength' from 'LeftCurly', because it is deprecated (since 6.10) and defaults to 80 anyway. --- java/metrics_checks.xml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/java/metrics_checks.xml b/java/metrics_checks.xml index e3b04ca..858d4ea 100644 --- a/java/metrics_checks.xml +++ b/java/metrics_checks.xml @@ -18,6 +18,8 @@

- Added a check for unused imports [UnusedImports].

+ - Limit line length to 80 [LineLength]. + Checkstyle is very configurable. Be sure to read the documentation at http://checkstyle.sf.net (or in your downloaded distribution).

@@ -37,7 +39,7 @@ <module name="FileTabCharacter"> <property name="eachLine" value="true"/> </module> - + <module name="SuppressWarningsFilter" /> <module name="TreeWalker"> <module name="OuterTypeFilename"/> @@ -52,7 +54,7 @@ <property name="allowNonPrintableEscapes" value="true"/> </module> <module name="LineLength"> - <property name="max" value="100"/> + <property name="max" value="80"/> <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/> </module> <module name="AvoidStarImport"/> @@ -64,9 +66,7 @@ <property name="tokens" value="LITERAL_TRY, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE, LITERAL_SWITCH"/> </module> <module name="NeedBraces"/> - <module name="LeftCurly"> - <property name="maxLineLength" value="100"/> - </module> + <module name="LeftCurly"/> <module name="RightCurly"/> <module name="RightCurly"> <property name="option" value="alone"/>