[hg] main-silver: Warnings elimination: @Override annotations an...

  • From:
  • To:
  • Subject: [hg] main-silver: Warnings elimination: @Override annotations an...
  • Date: Sun, 31 Mar 2013 04:41:21 -0700

changeset b6831103cbbb in main-silver ((none))
details: http://hg.netbeans.org/main-silver/rev/b6831103cbbb
description:
        Warnings elimination: @Override annotations and brackets.

diffstat:

 
debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractObjectVariable.java
 |  108 +++++++--
 
debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractVariable.java
       |   88 +++++--
 2 files changed, 136 insertions(+), 60 deletions(-)

diffs (595 lines):

diff --git 
a/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractObjectVariable.java
 
b/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractObjectVariable.java
--- 
a/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractObjectVariable.java
+++ 
b/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractObjectVariable.java
@@ -184,9 +184,12 @@
     *
     * @return string representation of type of this variable.
     */
+    @Override
     public int getFieldsCount () {
         Value v = getInnerValue ();
-        if (v == null) return 0;
+        if (v == null) {
+            return 0;
+        }
         if (v instanceof ArrayReference) {
             return ArrayReferenceWrapper.length0((ArrayReference) v);
         } else {
@@ -206,9 +209,12 @@
      *
      * @return field defined in this object
      */
+    @Override
     public Field getField (String name) {
         Value v = getInnerValue();
-        if (v == null) return null;
+        if (v == null) {
+            return null;
+        }
         com.sun.jdi.Field f;
         try {
             f = ReferenceTypeWrapper.fieldByName((ReferenceType) 
ValueWrapper.type(v), name);
@@ -221,7 +227,9 @@
         } catch (VMDisconnectedExceptionWrapper e) {
             return null;
         }
-        if (f == null) return null;
+        if (f == null) {
+            return null;
+        }
         return this.getField (
             f, 
             (ObjectReference) getInnerValue (),
@@ -233,15 +241,20 @@
      * Returns all fields declared in this type that are in interval
      * &lt;<code>from</code>, <code>to</code>).
      */
+    @Override
     public Field[] getFields (int from, int to) {
         Value v = getInnerValue ();
-        if (v == null) return new Field[] {};
+        if (v == null) {
+            return new Field[] {};
+        }
         try {
             if (v instanceof ArrayReference && (from > 0 || to < 
ArrayReferenceWrapper.length((ArrayReference) v))) {
                 // compute only requested elements
                 Type type = ValueWrapper.type(v);
                 ReferenceType rt = (ReferenceType) type;
-                if (to == 0) to = 
ArrayReferenceWrapper.length((ArrayReference) v);
+                if (to == 0) {
+                    to = ArrayReferenceWrapper.length((ArrayReference) v);
+                }
                 Field[] elements = getFieldsOfArray (
                         (ArrayReference) v, 
                         ArrayTypeWrapper.componentTypeName((ArrayType) rt),
@@ -271,6 +284,7 @@
      *
      * @return all static fields
      */
+    @Override
     public Field[] getAllStaticFields (int from, int to) {
         Value v = getInnerValue ();
         if (v == null || v instanceof ArrayReference) {
@@ -289,6 +303,7 @@
      * 
      * @return all inherited fields
      */
+    @Override
     public Field[] getInheritedFields (int from, int to) {
         Value v = getInnerValue ();
         if (v == null || v instanceof ArrayReference) {
@@ -317,6 +332,7 @@
         
     }
 
+    @Override
     public Super getSuper () {
         synchronized (superClassLoaded) {
             if (superClassLoaded[0]) {
@@ -369,8 +385,9 @@
     }
 
     public boolean hasAllTypes() {
-        if (getInnerValue () == null)
+        if (getInnerValue () == null) {
             return true;
+        }
         Type t;
         synchronized (valueTypeLoaded) {
             if (!valueTypeLoaded[0]) {
@@ -401,12 +418,14 @@
     }
     
     public List<JPDAClassType> getAllInterfaces() {
-        if (getInnerValue () == null)
+        if (getInnerValue () == null) {
             return null;
+        }
         try {
             Type t = getCachedType();
-            if (!(t instanceof ClassType))
+            if (!(t instanceof ClassType)) {
                 return null;
+            }
             ClassType ct = (ClassType) t;
             return getDebugger().getAllInterfaces(ct);
         } catch (ObjectCollectedExceptionWrapper ocex) {
@@ -424,6 +443,7 @@
      *
      * @return toString () value of this instance
      */
+    @Override
     public String getToStringValue () throws InvalidExpressionException {
         Value v = getInnerValue ();
         return getToStringValue(v, getDebugger(), 0);
@@ -449,7 +469,9 @@
     }
 
     static String getToStringValue (Value v, JPDADebuggerImpl debugger, int 
maxLength) throws InvalidExpressionException {
-        if (v == null) return null;
+        if (v == null) {
+            return null;
+        }
         com.sun.jdi.Type type;
         try {
             type = ValueWrapper.type(v);
@@ -464,13 +486,17 @@
     }
     
     static String getToStringValue (Value v, com.sun.jdi.Type type, 
JPDADebuggerImpl debugger, int maxLength) throws InvalidExpressionException {
-        if (v == null) return null;
+        if (v == null) {
+            return null;
+        }
         try {
-            if (!(type instanceof ClassType))
+            if (!(type instanceof ClassType)) {
                 return AbstractVariable.getValue (v);
+            }
             ClassType ct = (ClassType) type;
-            if (v instanceof CharValue)
+            if (v instanceof CharValue) {
                 return "\'" + v.toString () + "\'";
+            }
             /*JPDAThread ct = debugger.getCurrentThread();
             if (ct != null) { For the case that toString() should be blocked 
by pending actions
                 Object pendingAction = ((JPDAThreadImpl) 
ct).getPendingAction();
@@ -542,6 +568,7 @@
      *
      * @return value of given method call on this instance
      */
+    @Override
     public Variable invokeMethod (
         String methodName,
         String signature,
@@ -571,34 +598,41 @@
              
             // 1) find corrent method
             Value v = this.getInnerValue ();
-            if (v == null) return null;
+            if (v == null) {
+                return null;
+            }
             ClassType ct = (ClassType) getCachedType();
             Method method = null;
-            if (signature != null)
+            if (signature != null) {
                 method = ClassTypeWrapper.concreteMethodByName(
                         ct,
                         methodName, signature);
-            else {
+            } else {
                 List l = ReferenceTypeWrapper.methodsByName(
                         ct,
                         methodName);
                 int j, jj = l.size ();
-                for (j = 0; j < jj; j++)
+                for (j = 0; j < jj; j++) {
                     if ( !MethodWrapper.isAbstract((Method) l.get (j)) &&
-                         MethodWrapper.argumentTypeNames((Method) l.get 
(j)).size () == 0
+                         MethodWrapper.argumentTypeNames((Method) l.get 
(j)).isEmpty()
                     ) {
                         method = (Method) l.get (j);
                         break;
                     }
             }
+            }
             
             // 2) method not found => print all method signatures
             if (method == null) {
                 List l = ReferenceTypeWrapper.methodsByName(
                         ct, methodName);
                 int j, jj = l.size ();
-                for (j = 0; j < jj; j++)
-                    System.out.println 
(TypeComponentWrapper.signature((Method) l.get (j)));
+                logger.log(Level.INFO, "No method {0}.{1} with signature 
{2}",
+                           new Object[]{ TypeWrapper.name(ct), methodName, 
signature });
+                logger.info("Found following methods with signatures:");
+                for (j = 0; j < jj; j++) {
+                    logger.info (TypeComponentWrapper.signature((Method) 
l.get (j)));
+                }
                 throw new NoSuchMethodException (
                     TypeWrapper.name(ct) + "." +
                         methodName + " : " + signature
@@ -608,8 +642,9 @@
             // 3) call this method
             Value[] vs = new Value [arguments.length];
             int i, k = arguments.length;
-            for (i = 0; i < k; i++)
+            for (i = 0; i < k; i++) {
                 vs [i] = ((AbstractVariable) arguments [i]).getInnerValue ();
+            }
             v = getDebugger().invokeMethod (
                 (JPDAThreadImpl) thread,
                 (ObjectReference) v,
@@ -618,12 +653,13 @@
             );
             
             // 4) encapsulate result
-            if (v instanceof ObjectReference)
+            if (v instanceof ObjectReference) {
                 return new AbstractObjectVariable ( // It's also 
ObjectVariable
                         getDebugger(),
                         (ObjectReference) v,
                         getID() + method + "^"
                     );
+            }
             return new AbstractVariable (getDebugger(), v, getID() + method);
         } catch (InternalExceptionWrapper ex) {
             return null;
@@ -715,7 +751,9 @@
      */
     @Override
     public String getType () {
-        if (genericType != null) return genericType;
+        if (genericType != null) {
+            return genericType;
+        }
         try {
             String[] typeNamePtr = new String[1];
             getCachedType(typeNamePtr);
@@ -826,7 +864,7 @@
         }
         case 'L':
         {
-            StringBuffer typeName = new StringBuffer(50);
+            StringBuilder typeName = new StringBuilder(50);
             for (;;) {
                 c = signature.read();
                 if (c == ';') {
@@ -836,13 +874,17 @@
                 }
                 else if (c == '<') {
                     int idx = typeName.lastIndexOf("/");
-                    if (idx != -1) typeName.delete(0, idx + 1);
+                    if (idx != -1) {
+                        typeName.delete(0, idx + 1);
+                    }
                     typeName.append("<");
                     for (;;) {
                         String td = getTypeDescription(signature);
                         typeName.append(td);
                         c = signature.read();
-                        if (c == '>') break;
+                        if (c == '>') {
+                            break;
+                        }
                         signature.unread(c);
                         typeName.append(',');
                     }
@@ -858,7 +900,7 @@
     }
 
     private static String brackets (int arrayCount) {
-        StringBuffer sb = new StringBuffer (arrayCount * 2);
+        StringBuilder sb = new StringBuilder (arrayCount * 2);
         do {
             sb.append ("[]");
         } while (--arrayCount > 0);
@@ -978,15 +1020,16 @@
             for (i = 0; i < k; i++) {
                 com.sun.jdi.Field f = l.get (i);
                 Field field = this.getField (f, or, this.getID());
-                if (TypeComponentWrapper.isStatic(f))
+                if (TypeComponentWrapper.isStatic(f)) {
                     classStaticFields.add(field);
-                else {
-                    if (s.contains (f))
+                } else {
+                    if (s.contains (f)) {
                         classFields.add(field);
-                    else
+                    } else {
                         allInheretedFields.add(field);
                 }
             }
+            }
         } catch (ClassNotPreparedExceptionWrapper e) {
         } catch (InternalExceptionWrapper e) {
         } catch (VMDisconnectedExceptionWrapper e) {
@@ -1030,6 +1073,7 @@
         }
     }
     
+    @Override
     public List<ObjectVariable> getReferringObjects(long maxReferrers) {
         Value v = getJDIValue();
         if (v instanceof ObjectReference) {
@@ -1045,6 +1089,7 @@
                     return Collections.emptyList();
                 }
                 return new AbstractList<ObjectVariable>() {
+                    @Override
                     public ObjectVariable get(int i) {
                         ObjectReference obj = referrers.get(i);
                         if (obj instanceof ClassObjectReference) {
@@ -1055,6 +1100,7 @@
                         }
                     }
 
+                    @Override
                     public int size() {
                         return referrers.size();
                     }
@@ -1064,6 +1110,7 @@
         }
     }
     
+    @Override
     public long getUniqueID() {
         Value value = getJDIValue();
         if (!(value instanceof ObjectReference)) { // null or anything else 
than Object
@@ -1175,6 +1222,7 @@
     
     private class DebuggetStateListener extends Object implements 
PropertyChangeListener {
         
+        @Override
         public void propertyChange(PropertyChangeEvent evt) {
             if (JPDADebugger.PROP_STATE.equals(evt.getPropertyName())) {
                 Object newValue = evt.getNewValue();
diff --git 
a/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractVariable.java
 
b/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractVariable.java
--- 
a/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractVariable.java
+++ 
b/debugger.jpda/src/org/netbeans/modules/debugger/jpda/models/AbstractVariable.java
@@ -46,34 +46,26 @@
 
 import com.sun.jdi.ArrayReference;
 import com.sun.jdi.BooleanType;
-import com.sun.jdi.BooleanValue;
 import com.sun.jdi.ByteType;
-import com.sun.jdi.ByteValue;
 import com.sun.jdi.CharType;
 import com.sun.jdi.CharValue;
 import com.sun.jdi.ClassNotLoadedException;
 import com.sun.jdi.ClassObjectReference;
 import com.sun.jdi.ClassType;
 import com.sun.jdi.DoubleType;
-import com.sun.jdi.DoubleValue;
 import com.sun.jdi.Field;
 import com.sun.jdi.FloatType;
-import com.sun.jdi.FloatValue;
 import com.sun.jdi.IncompatibleThreadStateException;
 import com.sun.jdi.IntegerType;
-import com.sun.jdi.IntegerValue;
 import com.sun.jdi.InternalException;
 import com.sun.jdi.InvalidTypeException;
 import com.sun.jdi.InvocationException;
 import com.sun.jdi.LongType;
-import com.sun.jdi.LongValue;
-import com.sun.jdi.ObjectCollectedException;
 import com.sun.jdi.ObjectReference;
 import com.sun.jdi.PrimitiveType;
 import com.sun.jdi.PrimitiveValue;
 import com.sun.jdi.ReferenceType;
 import com.sun.jdi.ShortType;
-import com.sun.jdi.ShortValue;
 import com.sun.jdi.StringReference;
 import com.sun.jdi.Type;
 import com.sun.jdi.Value;
@@ -110,7 +102,6 @@
 import org.netbeans.modules.debugger.jpda.jdi.VMDisconnectedExceptionWrapper;
 import org.netbeans.modules.debugger.jpda.jdi.ValueWrapper;
 import org.netbeans.modules.debugger.jpda.jdi.VirtualMachineWrapper;
-import org.openide.util.Exceptions;
 import org.openide.util.NbBundle;
 
 
@@ -125,7 +116,7 @@
     private JPDADebuggerImpl debugger;
     private String          id;
     
-    private Set<PropertyChangeListener> listeners = new 
HashSet<PropertyChangeListener>();
+    private final Set<PropertyChangeListener> listeners = new 
HashSet<PropertyChangeListener>();
 
     
     public AbstractVariable (
@@ -136,9 +127,10 @@
         this.debugger = debugger;
         this.value = value;
         this.id = id;
-        if (this.id == null)
+        if (this.id == null) {
             this.id = Integer.toString(super.hashCode());
     }
+    }
 
     
     // public interface 
........................................................
@@ -148,18 +140,25 @@
     *
     * @return string representation of type of this variable.
     */
+    @Override
     public String getValue () {
         Value v = getInnerValue ();
         return getValue(v);
     }
     
     static String getValue (Value v) {
-        if (v == null) return "null";
-        if (v instanceof VoidValue) return "void";
-        if (v instanceof CharValue)
+        if (v == null) {
+            return "null";
+        }
+        if (v instanceof VoidValue) {
+            return "void";
+        }
+        if (v instanceof CharValue) {
             return "\'" + v.toString () + "\'";
-        if (v instanceof PrimitiveValue)
+        }
+        if (v instanceof PrimitiveValue) {
             return v.toString ();
+        }
         try {
             if (v instanceof StringReference) {
                 String str = StringReferenceWrapper.value((StringReference) 
v);
@@ -168,11 +167,13 @@
                 }
                 return "\"" + str + "\"";
             }
-            if (v instanceof ClassObjectReference)
+            if (v instanceof ClassObjectReference) {
                 return "class " + 
ReferenceTypeWrapper.name(ClassObjectReferenceWrapper.reflectedType((ClassObjectReference)
 v));
-            if (v instanceof ArrayReference)
+            }
+            if (v instanceof ArrayReference) {
                 return "#" + 
ObjectReferenceWrapper.uniqueID((ArrayReference) v) +
                     "(length=" + 
ArrayReferenceWrapper.length((ArrayReference) v) + ")";
+            }
             return "#" + ObjectReferenceWrapper.uniqueID((ObjectReference) 
v);
         } catch (InternalExceptionWrapper iex) {
             return "";
@@ -291,14 +292,30 @@
                 PrimitiveValue pv = (PrimitiveValue) value;
                 try {
                     VirtualMachine vm = MirrorWrapper.virtualMachine(pv);
-                    if (type instanceof BooleanType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.booleanValue(pv));
-                    if (type instanceof ByteType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.byteValue(pv));
-                    if (type instanceof CharType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.charValue(pv));
-                    if (type instanceof ShortType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.shortValue(pv));
-                    if (type instanceof IntegerType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.intValue(pv));
-                    if (type instanceof LongType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.longValue(pv));
-                    if (type instanceof FloatType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.floatValue(pv));
-                    if (type instanceof DoubleType) return 
VirtualMachineWrapper.mirrorOf(vm, PrimitiveValueWrapper.doubleValue(pv));
+                    if (type instanceof BooleanType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.booleanValue(pv));
+                    }
+                    if (type instanceof ByteType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.byteValue(pv));
+                    }
+                    if (type instanceof CharType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.charValue(pv));
+                    }
+                    if (type instanceof ShortType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.shortValue(pv));
+                    }
+                    if (type instanceof IntegerType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.intValue(pv));
+                    }
+                    if (type instanceof LongType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.longValue(pv));
+                    }
+                    if (type instanceof FloatType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.floatValue(pv));
+                    }
+                    if (type instanceof DoubleType) {
+                        return VirtualMachineWrapper.mirrorOf(vm, 
PrimitiveValueWrapper.doubleValue(pv));
+                    }
                 } catch (InternalExceptionWrapper e) {
                 } catch (VMDisconnectedExceptionWrapper e) {
                 }
@@ -332,6 +349,7 @@
         throw new InternalError ();
     }
     
+    @Override
     public void setObject(Object bean) {
         try {
             if (bean instanceof String) {
@@ -342,9 +360,7 @@
                 throw new IllegalArgumentException(""+bean);
             }
         } catch (InvalidExpressionException ieex) {
-            IllegalArgumentException iaex = new 
IllegalArgumentException(ieex.getLocalizedMessage());
-            iaex.initCause(ieex);
-            throw iaex;
+            throw new IllegalArgumentException(ieex);
         }
     }
 
@@ -353,8 +369,11 @@
      *
      * @return declared type of this local
      */
+    @Override
     public String getType () {
-        if (getInnerValue () == null) return "";
+        if (getInnerValue () == null) {
+            return "";
+        }
         try {
             return TypeWrapper.name(ValueWrapper.type(this.getInnerValue()));
         } catch (InternalExceptionWrapper ex) {
@@ -370,7 +389,9 @@
     
     public JPDAClassType getClassType() {
         Value value = getInnerValue();
-        if (value == null) return null;
+        if (value == null) {
+            return null;
+        }
         com.sun.jdi.Type type;
         try {
             type = ValueWrapper.type(value);
@@ -388,11 +409,13 @@
         }
     }
     
+    @Override
     public boolean equals (Object o) {
         return  (o instanceof AbstractVariable) &&
                 (id.equals (((AbstractVariable) o).id));
     }
     
+    @Override
     public int hashCode () {
         return id.hashCode ();
     }
@@ -420,6 +443,7 @@
         //getModel ().fireTableValueChangedChanged (this, null);
     }
     
+    @Override
     public Value getJDIValue() {
         return getInnerValue();
     }
@@ -434,23 +458,27 @@
     
     private int cloneNumber = 1;
     
+    @Override
     public Variable clone() {
         AbstractVariable clon = new AbstractVariable(debugger, value, id + 
"_clone"+(cloneNumber++));
         return clon;
     }
     
+    @Override
     public final void addPropertyChangeListener(PropertyChangeListener l) {
         synchronized (listeners) {
             listeners.add(l);
         }
     }
     
+    @Override
     public final void removePropertyChangeListener(PropertyChangeListener l) 
{
         synchronized (listeners) {
             listeners.remove(l);
         }
     }
     
+    @Override
     public String toString () {
         return "Variable ";
     }

[hg] main-silver: Warnings elimination: @Override annotations an...

mentlicher 03/31/2013

Project Features

About this Project

Debugger was started in November 2009, is owned by tpavek, and has 29 members.
By use of this website, you agree to the NetBeans Policies and Terms of Use (revision 20140418.2d69abc). © 2013, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo
 
 
Close
loading
Please Confirm
Close