Fix quality flaws

diff --git a/sonar-core/src/main/java/org/sonar/core/profiling/LoggingWatch.java b/sonar-core/src/main/java/org/sonar/core/profiling/LoggingWatch.java
index 18112bcf8a24a8c7be48ca33cca55c0e0c3f5e0a..d0ef0c47510385e3c82129a2fe8636e49d9346cd 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/profiling/LoggingWatch.java
+++ b/sonar-core/src/main/java/org/sonar/core/profiling/LoggingWatch.java
@@ -23,7 +23,7 @@ import com.google.common.annotations.VisibleForTesting;
 import org.slf4j.Logger;
 import org.sonar.api.utils.System2;
 
-class LoggingWatch extends StopWatch {
+class LoggingWatch implements StopWatch {
 
   private Logger logger;
   private System2 system;

diff --git a/sonar-core/src/main/java/org/sonar/core/profiling/NoopWatch.java b/sonar-core/src/main/java/org/sonar/core/profiling/NoopWatch.java
index 419012500b88b6d978a1ba6e23e9463eff84bf6f..61b9a30271ca19355e43b68d73b924fee969cbe7 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/profiling/NoopWatch.java
+++ b/sonar-core/src/main/java/org/sonar/core/profiling/NoopWatch.java
@@ -20,7 +20,7 @@
 package org.sonar.core.profiling;
 
 
-class NoopWatch extends StopWatch {
+class NoopWatch implements StopWatch {
 
   @Override
   public void stop(String message, Object... args) {

diff --git a/sonar-core/src/main/java/org/sonar/core/profiling/StopWatch.java b/sonar-core/src/main/java/org/sonar/core/profiling/StopWatch.java
index 2c2b613811dea0f955ea46c90c672799fe3a83b2..cd9ca655bb5ee933814463c6c94d54a353feed66 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/profiling/StopWatch.java
+++ b/sonar-core/src/main/java/org/sonar/core/profiling/StopWatch.java
@@ -19,7 +19,10 @@
  */
 package org.sonar.core.profiling;
 
-public abstract class StopWatch {
+public interface StopWatch {
 
-  public abstract void stop(String message, Object... args);
+  /**
+   * Stop the watch and print provided profiling message
+   */
+  void stop(String message, Object... args);
 }

diff --git a/sonar-markdown/src/main/java/org/sonar/markdown/HtmlBlockquoteChannel.java b/sonar-markdown/src/main/java/org/sonar/markdown/HtmlBlockquoteChannel.java
index 286e7e58c809e101d14fd77fad0dd25b0c9be1b4..da1594d9a306302c205545175a5130367dbfa759 100644 (file)
--- a/sonar-markdown/src/main/java/org/sonar/markdown/HtmlBlockquoteChannel.java
+++ b/sonar-markdown/src/main/java/org/sonar/markdown/HtmlBlockquoteChannel.java
@@ -47,14 +47,12 @@ class HtmlBlockquoteChannel extends Channel<MarkdownOutput> {
   @Override
   public boolean consume(CodeReader code, MarkdownOutput output) {
     try {
-      if (code.getColumnPosition() == 0) {
-        if (quotedLineElement.consume(code, output)) {
-          while (endOfLine.consume(code, output) && quotedLineElement.consume(code, output)) {
-            // consume input
-          }
-          output.append("</blockquote>");
-          return true;
+      if (code.getColumnPosition() == 0 && quotedLineElement.consume(code, output)) {
+        while (endOfLine.consume(code, output) && quotedLineElement.consume(code, output)) {
+          // consume input
         }
+        output.append("</blockquote>");
+        return true;
       }
       return false;
     } finally {
@@ -79,15 +77,18 @@ class HtmlBlockquoteChannel extends Channel<MarkdownOutput> {
     }
 
     private int searchIndexOfFirstCharacter(CharSequence token) {
-      for (int index = 0; index < token.length(); index++) {
+      int index = 0;
+      while (index < token.length()) {
         if (token.charAt(index) == '&') {
           index += 4;
-          while (++ index < token.length()) {
+          while (index < token.length()) {
+            index ++;
             if (token.charAt(index) != ' ') {
               return index;
             }
           }
         }
+        index ++;
       }
       return token.length() - 1;
     }