Cleanup (formatting)
authorRoman Shevchenko <roman.shevchenko@jetbrains.com>
Wed, 28 Oct 2015 15:17:59 +0000 (16:17 +0100)
committerRoman Shevchenko <roman.shevchenko@jetbrains.com>
Wed, 28 Oct 2015 15:35:58 +0000 (16:35 +0100)
platform/platform-api/src/com/intellij/util/ui/ButtonlessScrollBarUI.java
platform/util-rt/src/com/intellij/openapi/util/Comparing.java

index 51aed0ba748568d195b5139cfc6ad8f66a8c9d09..00892e4c57e8a57c22428a09b0310fa2876b448d 100644 (file)
@@ -291,7 +291,8 @@ public class ButtonlessScrollBarUI extends BasicScrollBarUI {
   public void layoutContainer(Container scrollbarContainer) {
     try {
       super.layoutContainer(scrollbarContainer);
-    } catch (NullPointerException ignore) {
+    }
+    catch (NullPointerException ignore) {
       //installUI is not performed yet or uninstallUI has set almost every field to null. Just ignore it //IDEA-89674
     }
   }
@@ -316,9 +317,7 @@ public class ButtonlessScrollBarUI extends BasicScrollBarUI {
       // We want to repaint whole scrollbar even if thumb wasn't moved (on small scroll of a big panel)
       // Even if scrollbar wasn't changed itself, myRepaintCallback could need repaint
 
-        /* Update thumbRect, and repaint the union of x,y,w,h and
-         * the old thumbRect.
-         */
+      // Update thumbRect, and repaint the union of x,y,w,h and the old thumbRect.
       int minX = Math.min(x, trackRect.x);
       int minY = Math.min(y, trackRect.y);
       int maxX = Math.max(x + width, trackRect.x + trackRect.width);
@@ -327,13 +326,11 @@ public class ButtonlessScrollBarUI extends BasicScrollBarUI {
       thumbRect.setBounds(x, y, width, height);
       scrollbar.repaint(minX, minY, maxX - minX, maxY - minY);
 
-      // Once there is API to determine the mouse location this will need
-      // to be changed.
+      // Once there is API to determine the mouse location this will need to be changed.
       setThumbRollover(false);
     }
   }
 
-
   @Override
   protected ModelListener createModelListener() {
     return new ModelListener() {
index cc5c4ab30b1e702834cc8760090174756283b174..cd69bb2a59a24499f32b6ca8bea9ed8025b4650f 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2014 JetBrains s.r.o.
+ * Copyright 2000-2015 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,12 +27,12 @@ import java.util.*;
 public class Comparing {
   private Comparing() { }
 
-  public static <T> boolean equal(@Nullable T arg1, @Nullable T arg2){
+  public static <T> boolean equal(@Nullable T arg1, @Nullable T arg2) {
     if (arg1 == arg2) return true;
-    if (arg1 == null || arg2 == null){
+    if (arg1 == null || arg2 == null) {
       return false;
     }
-    if (arg1 instanceof Object[] && arg2 instanceof Object[]){
+    if (arg1 instanceof Object[] && arg2 instanceof Object[]) {
       Object[] arr1 = (Object[])arg1;
       Object[] arr2 = (Object[])arg2;
       return Arrays.equals(arr1, arr2);
@@ -43,8 +43,8 @@ public class Comparing {
     return arg1.equals(arg2);
   }
 
-  public static <T> boolean equal(@Nullable T[] arr1, @Nullable T[] arr2){
-    if (arr1 == null || arr2 == null){
+  public static <T> boolean equal(@Nullable T[] arr1, @Nullable T[] arr2) {
+    if (arr1 == null || arr2 == null) {
       return arr1 == arr2;
     }
     return Arrays.equals(arr1, arr2);
@@ -82,20 +82,20 @@ public class Comparing {
     return true;
   }
 
-  public static boolean equal(@Nullable String arg1, @Nullable String arg2, boolean caseSensitive){
-    if (arg1 == null || arg2 == null){
+  public static boolean equal(@Nullable String arg1, @Nullable String arg2, boolean caseSensitive) {
+    if (arg1 == null || arg2 == null) {
       return arg1 == arg2;
     }
-    else{
+    else {
       return caseSensitive ? arg1.equals(arg2) : arg1.equalsIgnoreCase(arg2);
     }
   }
 
-  public static boolean strEqual(@Nullable String arg1, @Nullable String arg2){
+  public static boolean strEqual(@Nullable String arg1, @Nullable String arg2) {
     return strEqual(arg1, arg2, true);
   }
 
-  public static boolean strEqual(@Nullable String arg1, @Nullable String arg2, boolean caseSensitive){
+  public static boolean strEqual(@Nullable String arg1, @Nullable String arg2, boolean caseSensitive) {
     return equal(arg1 == null ? "" : arg1, arg2 == null ? "" : arg2, caseSensitive);
   }
 
@@ -110,6 +110,7 @@ public class Comparing {
         return false;
       }
     }
+
     return true;
   }
 
@@ -128,11 +129,18 @@ public class Comparing {
         return false;
       }
     }
+
     return true;
   }
 
-  public static int hashcode(@Nullable Object obj) { return obj == null ? 0 : obj.hashCode(); }
-  public static int hashcode(Object obj1, Object obj2) { return hashcode(obj1) ^ hashcode(obj2); }
+  @SuppressWarnings("MethodNamesDifferingOnlyByCase")
+  public static int hashcode(@Nullable Object obj) {
+    return obj == null ? 0 : obj.hashCode();
+  }
+
+  public static int hashcode(Object obj1, Object obj2) {
+    return hashcode(obj1) ^ hashcode(obj2);
+  }
 
   public static int compare(byte o1, byte o2) {
     return o1 < o2 ? -1 : o1 == o2 ? 0 : 1;
@@ -149,6 +157,7 @@ public class Comparing {
   public static int compare(long o1, long o2) {
     return o1 < o2 ? -1 : o1 == o2 ? 0 : 1;
   }
+
   public static int compare(double o1, double o2) {
     return o1 < o2 ? -1 : o1 == o2 ? 0 : 1;
   }
@@ -165,16 +174,17 @@ public class Comparing {
       if (o1[i] > o2[i]) return 1;
       else if (o1[i] < o2[i]) return -1;
     }
+
     return 0;
   }
 
-  public static <T extends Comparable<T>> int compare(@Nullable final T o1, @Nullable final T o2) {
+  public static <T extends Comparable<T>> int compare(@Nullable T o1, @Nullable T o2) {
     if (o1 == null) return o2 == null ? 0 : -1;
     if (o2 == null) return 1;
     return o1.compareTo(o2);
   }
 
-  public static <T> int compare(@Nullable final T o1, @Nullable final T o2, @NotNull final Comparator<T> notNullComparator) {
+  public static <T> int compare(@Nullable T o1, @Nullable T o2, @NotNull Comparator<T> notNullComparator) {
     if (o1 == null) return o2 == null ? 0 : -1;
     if (o2 == null) return 1;
     return notNullComparator.compare(o1, o2);