#709: fixes after review

yegor256 · May 31, 2016 · db7d9de · db7d9de
1 parent e488c53
commit db7d9de
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/...checkstyle/src/main/java/com/qulice/checkstyle/FinalSemicolonInTryWithResourcesCheck.java b/...checkstyle/src/main/java/com/qulice/checkstyle/FinalSemicolonInTryWithResourcesCheck.java
@@ -34,7 +34,9 @@
 import com.puppycrawl.tools.checkstyle.api.TokenTypes;
 
 /**
- * Checks that try-with-resources does not end with a semicolon.
+ * Checks that try-with-resources does not end with a semicolon. Implementation
+ * relies on existence of semicolon inside of RESOURCE_SPECIFICATION token
+ * as interpreted by Checkstyle.
  *
  * @author Hamdi Douss (douss.hamdi@gmail.com)
  * @version $Id$

diff --git a/qulice-checkstyle/src/main/java/com/qulice/checkstyle/MethodsOrderCheck.java b/qulice-checkstyle/src/main/java/com/qulice/checkstyle/MethodsOrderCheck.java
@@ -82,7 +82,7 @@ private void checkClass(final DetailAST node) {
     /**
      * Checks order of methods.
      *
-     * @param methods Iterable&lt;DetailAST&gt; methods
+     * @param methods @code{Iterable<DetailAST>} methods
      */
     private void checkOrder(final Iterable<DetailAST> methods) {
         MethodsOrderCheck.Modifiers prev = MethodsOrderCheck.Modifiers.PUB;

diff --git a/qulice-checkstyle/src/test/java/com/qulice/checkstyle/CheckstyleValidatorTest.java b/qulice-checkstyle/src/test/java/com/qulice/checkstyle/CheckstyleValidatorTest.java
@@ -698,7 +698,7 @@ public void rejectsNonDiamondOperatorUsage() throws Exception {
      * CheckstyleValidator can allow diamond operator usage.
      * @throws Exception If error
      * @todo #715:30min add test for next situation
-     *  `return new ArrayList&lt;String&gt;();`
+     *  `return @code{new ArrayList<String>();}`
      */
     @Test
     public void allowsDiamondOperatorUsage() throws Exception {