From: Julien HENRY Date: Tue, 24 Feb 2015 14:28:15 +0000 (+0100) Subject: Fix some quality flaws X-Git-Tag: 5.1-RC1~74 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=98c4b34bade9af878d170bd429ac4b3207b930b4;p=sonarqube.git Fix some quality flaws --- diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/fs/internal/DefaultInputDir.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/fs/internal/DefaultInputDir.java index ac48ea2b3b0..8979e1f3468 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/fs/internal/DefaultInputDir.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/fs/internal/DefaultInputDir.java @@ -83,7 +83,7 @@ public class DefaultInputDir implements InputDir { if (this == o) { return true; } - if (!(this.getClass() == o.getClass())) { + if (o == null || this.getClass() != o.getClass()) { return false; } diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/highlighting/internal/DefaultHighlighting.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/highlighting/internal/DefaultHighlighting.java index ca1a0b4a355..60e0db9a2d3 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/highlighting/internal/DefaultHighlighting.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/highlighting/internal/DefaultHighlighting.java @@ -20,7 +20,6 @@ package org.sonar.api.batch.sensor.highlighting.internal; import com.google.common.base.Preconditions; -import com.google.common.collect.Ordering; import com.google.common.collect.Sets; import org.sonar.api.batch.fs.InputFile; import org.sonar.api.batch.fs.internal.DefaultInputFile; @@ -31,6 +30,7 @@ import org.sonar.api.batch.sensor.internal.SensorStorage; import javax.annotation.Nullable; +import java.util.Comparator; import java.util.Iterator; import java.util.Set; @@ -45,10 +45,9 @@ public class DefaultHighlighting extends DefaultStorable implements NewHighlight public DefaultHighlighting(@Nullable SensorStorage storage) { super(storage); - syntaxHighlightingRuleSet = Sets.newTreeSet(new Ordering() { + syntaxHighlightingRuleSet = Sets.newTreeSet(new Comparator() { @Override - public int compare(@Nullable SyntaxHighlightingRule left, - @Nullable SyntaxHighlightingRule right) { + public int compare(SyntaxHighlightingRule left, SyntaxHighlightingRule right) { int result = left.getStartPosition() - right.getStartPosition(); if (result == 0) { result = right.getEndPosition() - left.getEndPosition();