Fix quality flaws

diff --git a/server/sonar-server/src/main/java/org/sonar/server/ws/WebServicesWs.java b/server/sonar-server/src/main/java/org/sonar/server/ws/WebServicesWs.java
index 301214454fb778827cde7db27f795ace27ac1cf9..1eaaad67b080faf55f543f26b2b79d4e1a6ae49b 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/ws/WebServicesWs.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/ws/WebServicesWs.java
@@ -72,11 +72,11 @@ public class WebServicesWs implements WebService {
       .setDefaultValue("false");
   }
 
-  private void defineResponseExample(final Context context, NewController controller) {
+  private static void defineResponseExample(final Context context, NewController controller) {
     NewAction action = controller
       .createAction("response_example")
       .setDescription("Display web service response example")
-      .setResponseExample(getClass().getResource("response_example-example.json"))
+      .setResponseExample(WebServicesWs.class.getResource("response_example-example.json"))
       .setSince("4.4")
       .setHandler(new RequestHandler() {
         @Override
@@ -161,7 +161,7 @@ public class WebServicesWs implements WebService {
     }
   }
 
-  private void writeAction(JsonWriter writer, Action action, boolean includeInternals) {
+  private static void writeAction(JsonWriter writer, Action action, boolean includeInternals) {
     if (includeInternals || !action.isInternal()) {
       writer.beginObject();
       writer.prop("key", action.key());

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/rule/Checks.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/rule/Checks.java
index 7f021639917331600fa45252f642242cc396274f..c223f418a248d8f938250d9c2f591c69004e6603 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/rule/Checks.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/rule/Checks.java
@@ -154,7 +154,7 @@ public class Checks<C> {
     return StringUtils.defaultIfEmpty(key, clazz.getCanonicalName());
   }
 
-  private Object instantiate(ActiveRule activeRule, Object checkClassOrInstance) {
+  private static Object instantiate(ActiveRule activeRule, Object checkClassOrInstance) {
     try {
       Object check = checkClassOrInstance;
       if (check instanceof Class) {

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/checks/AnnotationCheckFactory.java b/sonar-plugin-api/src/main/java/org/sonar/api/checks/AnnotationCheckFactory.java
index be86b62992e9bd8dfd647620067247b7d55a9d15..397965bac4bdbbcf09c2e757e8dd7d71a10e4dad 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/checks/AnnotationCheckFactory.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/checks/AnnotationCheckFactory.java
@@ -74,7 +74,7 @@ public final class AnnotationCheckFactory extends CheckFactory {
     return null;
   }
 
-  private Object instantiate(ActiveRule activeRule, Object checkClassOrInstance) {
+  private static Object instantiate(ActiveRule activeRule, Object checkClassOrInstance) {
     try {
       Object check = checkClassOrInstance;
       if (check instanceof Class) {

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/rules/AnnotationRuleParser.java b/sonar-plugin-api/src/main/java/org/sonar/api/rules/AnnotationRuleParser.java
index fd0df622783d578362ed7f6afdd42820a60fca6f..187f835d569a1b1ea8941e4af4b5f133b8f9d25c 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/rules/AnnotationRuleParser.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/rules/AnnotationRuleParser.java
@@ -55,7 +55,7 @@ public final class AnnotationRuleParser {
     return rules;
   }
 
-  private Rule create(String repositoryKey, Class annotatedClass) {
+  private static Rule create(String repositoryKey, Class annotatedClass) {
     org.sonar.check.Rule ruleAnnotation = AnnotationUtils.getAnnotation(annotatedClass, org.sonar.check.Rule.class);
     if (ruleAnnotation != null) {
       return toRule(repositoryKey, annotatedClass, ruleAnnotation);