(SONAR-3893) Improve the highlighter API to not depend on sonar-channel and allow to work on multi-line tokens - Updated API to reflect comments and feedback

2 files changed, 4 insertions, 4 deletions

diff --git a/sonar-core/src/main/java/org/sonar/core/source/DefaultHighlightable.java b/sonar-core/src/main/java/org/sonar/core/source/DefaultHighlightable.java
index 25435ee4dc1..faa6665e27c 100644
--- a/sonar-core/src/main/java/org/sonar/core/source/DefaultHighlightable.java
+++ b/sonar-core/src/main/java/org/sonar/core/source/DefaultHighlightable.java
@@ -28,12 +28,12 @@ public class DefaultHighlightable implements Highlightable {
   private final SyntaxHighlightingRuleSet.Builder highlightingRulesBuilder;
 
   public DefaultHighlightable() {
-    highlightingRulesBuilder = new SyntaxHighlightingRuleSet.Builder();
+    highlightingRulesBuilder = SyntaxHighlightingRuleSet.builder();
   }
 
   @Override
-  public SyntaxHighlightingRuleSet.Builder highlightText(int startOffset, int endOffset, String typeOfText) {
-    return highlightingRulesBuilder.registerHighlightingRule(startOffset, endOffset, typeOfText);
+  public void highlightText(int startOffset, int endOffset, String typeOfText) {
+    highlightingRulesBuilder.registerHighlightingRule(startOffset, endOffset, typeOfText);
   }
 
   @Override
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/scan/source/Highlightable.java b/sonar-plugin-api/src/main/java/org/sonar/api/scan/source/Highlightable.java
index ee1eaeb8191..4aaad9fe17f 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/scan/source/Highlightable.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/scan/source/Highlightable.java
@@ -23,5 +23,5 @@ import org.sonar.api.component.Perspective;
 
 public interface Highlightable extends Perspective {
 
-  SyntaxHighlightingRuleSet.Builder highlightText(int startOffset, int endOffset, String typeOfText);
+  void highlightText(int startOffset, int endOffset, String typeOfText);
 }