X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=utils%2Fcheckstyle%2Fcheckstyle.xml;h=85ac8e64263ae871bbab731c1a2694035c1ce132;hb=29433ea97308601bc17d76d376c6e629b0b28fa5;hp=7ca2d26276d543e4640892e918d576bbd481d5ca;hpb=78d4775e4450ba6579c5d9824a78ad99cbccda22;p=jalview.git

diff --git a/utils/checkstyle/checkstyle.xml b/utils/checkstyle/checkstyle.xml
index 7ca2d26..85ac8e6 100644
--- a/utils/checkstyle/checkstyle.xml
+++ b/utils/checkstyle/checkstyle.xml
@@ -27,14 +27,6 @@
 		<property name="checkFormat" value="$1"/>
 	</module>
 
-	<!--
-		Maximum line count for source files
-	-->
-	<module name="FileLength">
-		<property name="max" value="1200"/>
-		<property name="fileExtensions" value="java"/>
-	</module>
-
 	<!-- 
 		Check language bundles have the same keys and no duplicates
 		(ensure Checkstyle is configured to scan non-source files)
@@ -47,6 +39,15 @@
 	    <property name="fileExtensions" value="properties" />
 		<property name="severity" value="error"/>
 	</module>
+
+	<!--
+		Maximum line count for source files
+		(note this can't be inside TreeWalker)
+	-->
+	<module name="FileLength">
+		<property name="max" value="1200"/>
+		<property name="fileExtensions" value="java"/>
+	</module>
 	
 	<module name="TreeWalker">
 
@@ -429,12 +430,20 @@
 		<module name="HiddenField"/> 
 
 		<!-- 
-			Check that proper logging is used and never printing to System.out.
+			Check that proper logging is used.
 			This may be suppressed in the class that provides logging functions.
 		-->
 		<module name="RegexpSinglelineJava">
+			<property name="id" value="NoSysout" />
 			<property name="format" value="System\.out\.println"/>
 			<property name="ignoreComments" value="true"/>
+			<message key="regexp.exceeded" value="Should use jalview.bin.Cache.log for logging"/>
+		</module>
+		<module name="RegexpSinglelineJava">
+			<property name="id" value="NoSyserr" />
+			<property name="format" value="System\.err\.println"/>
+			<property name="ignoreComments" value="true"/>
+			<message key="regexp.exceeded" value="Should use jalview.bin.Cache.log for logging"/>
 		</module>
 
 		<!--
@@ -585,7 +594,13 @@
 			Check for TODO and similar comments 
 	 	-->
 		<module name="TodoComment">
-			<property name="format" value="(TODO)|(FIXME)|(DOCUMENT ME)"/>
+			<property name="format" value="(TODO)|(FIXME)"/>
+			<message key="todo.match" value="TODO or FIXME comment"/>
+		</module>
+
+		<module name="TodoComment">
+			<property name="format" value="DOCUMENT ME"/>
+			<message key="todo.match" value="Documentation incomplete"/>
 		</module>
 
 	</module>