aboutsummaryrefslogtreecommitdiffstats
path: root/sonar-colorizer/src
diff options
context:
space:
mode:
Diffstat (limited to 'sonar-colorizer/src')
-rw-r--r--sonar-colorizer/src/main/java/org/sonar/colorizer/HtmlDecorator.java17
-rw-r--r--sonar-colorizer/src/test/java/org/sonar/colorizer/HtmlDecoratorTest.java2
2 files changed, 8 insertions, 11 deletions
diff --git a/sonar-colorizer/src/main/java/org/sonar/colorizer/HtmlDecorator.java b/sonar-colorizer/src/main/java/org/sonar/colorizer/HtmlDecorator.java
index 0d0b4386118..5e4e2e512d8 100644
--- a/sonar-colorizer/src/main/java/org/sonar/colorizer/HtmlDecorator.java
+++ b/sonar-colorizer/src/main/java/org/sonar/colorizer/HtmlDecorator.java
@@ -30,7 +30,7 @@ public class HtmlDecorator extends Tokenizer {
private static final String CSS_PATH = "/sonar-colorizer.css";
private HtmlOptions options;
- private int lineId = 1;
+ private int lineId;
private static final int LF = (int) '\n';
private static final int CR = (int) '\r';
@@ -76,18 +76,15 @@ public class HtmlDecorator extends Tokenizer {
@Override
public boolean consume(CodeReader code, HtmlCodeBuilder codeBuilder) {
- int currentChar = code.peek();
-
- if (currentChar == LF) {
- codeBuilder.appendWithoutTransforming(getTagAfter());
- codeBuilder.appendWithoutTransforming(getTagBefore());
- }
-
- if (currentChar == LF || currentChar == CR) {
+ int lineNumber = code.getLinePosition();
+ if (code.peek() == LF || code.peek() == CR) {
code.pop();
+ if (lineNumber != code.getLinePosition()) {
+ codeBuilder.appendWithoutTransforming(getTagAfter());
+ codeBuilder.appendWithoutTransforming(getTagBefore());
+ }
return true;
}
-
return false;
}
diff --git a/sonar-colorizer/src/test/java/org/sonar/colorizer/HtmlDecoratorTest.java b/sonar-colorizer/src/test/java/org/sonar/colorizer/HtmlDecoratorTest.java
index a49dafe025b..916640de168 100644
--- a/sonar-colorizer/src/test/java/org/sonar/colorizer/HtmlDecoratorTest.java
+++ b/sonar-colorizer/src/test/java/org/sonar/colorizer/HtmlDecoratorTest.java
@@ -80,7 +80,7 @@ public class HtmlDecoratorTest {
public void shouldAddTagsBetweenEachLine() {
HtmlOptions options = new HtmlOptions().setGenerateTable(true).setGenerateHtmlHeader(false);
HtmlDecorator decorator = new HtmlDecorator(options);
- CodeReader code = new CodeReader("\n\r\n");
+ CodeReader code = new CodeReader("\r\n\r");
HtmlCodeBuilder output = new HtmlCodeBuilder();
output.appendWithoutTransforming(decorator.getTagBeginOfFile());