You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by jc...@apache.org on 2013/07/30 05:57:36 UTC

svn commit: r1508292 - /commons/proper/proxy/branches/version-2.0-work/core/src/main/java/org/apache/commons/proxy2/invoker/RecordedInvocation.java

Author: jcarman
Date: Tue Jul 30 03:57:36 2013
New Revision: 1508292

URL: http://svn.apache.org/r1508292
Log:
Fixing Sonar issues.

Modified:
    commons/proper/proxy/branches/version-2.0-work/core/src/main/java/org/apache/commons/proxy2/invoker/RecordedInvocation.java

Modified: commons/proper/proxy/branches/version-2.0-work/core/src/main/java/org/apache/commons/proxy2/invoker/RecordedInvocation.java
URL: http://svn.apache.org/viewvc/commons/proper/proxy/branches/version-2.0-work/core/src/main/java/org/apache/commons/proxy2/invoker/RecordedInvocation.java?rev=1508292&r1=1508291&r2=1508292&view=diff
==============================================================================
--- commons/proper/proxy/branches/version-2.0-work/core/src/main/java/org/apache/commons/proxy2/invoker/RecordedInvocation.java (original)
+++ commons/proper/proxy/branches/version-2.0-work/core/src/main/java/org/apache/commons/proxy2/invoker/RecordedInvocation.java Tue Jul 30 03:57:36 2013
@@ -24,51 +24,51 @@ import java.lang.reflect.Method;
 
 /**
  * Detached representation of a method invocation.
+ *
  * @author James Carman
  */
 public class RecordedInvocation
 {
-//**********************************************************************************************************************
+//----------------------------------------------------------------------------------------------------------------------
 // Fields
-//**********************************************************************************************************************
+//----------------------------------------------------------------------------------------------------------------------
 
     private final Method invokedMethod;
     private final Object[] arguments;
 
-  //**********************************************************************************************************************
- // Constructors
- //**********************************************************************************************************************
+//----------------------------------------------------------------------------------------------------------------------
+// Constructors
+//----------------------------------------------------------------------------------------------------------------------
 
     /**
      * Create a new RecordedInvocation instance.
+     *
      * @param invokedMethod
      * @param arguments
      */
-    public RecordedInvocation( Method invokedMethod, Object[] arguments )
+    public RecordedInvocation(Method invokedMethod, Object[] arguments)
     {
         this.invokedMethod = invokedMethod;
         this.arguments = ArrayUtils.clone(arguments);
     }
 
-  //**********************************************************************************************************************
- // Canonical Methods
- //**********************************************************************************************************************
+//----------------------------------------------------------------------------------------------------------------------
+// Getter/Setter Methods
+//----------------------------------------------------------------------------------------------------------------------
 
     /**
      * Get the invokedMethod.
+     *
      * @return Method
      */
-    public Method getInvokedMethod() {
+    public Method getInvokedMethod()
+    {
         return invokedMethod;
     }
 
-    /**
-     * Get the arguments.
-     * @return Object[]
-     */
-    public Object[] getArguments() {
-        return arguments;
-    }
+//----------------------------------------------------------------------------------------------------------------------
+// Canonical Methods
+//----------------------------------------------------------------------------------------------------------------------
 
     /**
      * {@inheritDoc}
@@ -81,10 +81,10 @@ public class RecordedInvocation
         buffer.append(invokedMethod.getName());
         buffer.append("(");
         int count = arguments.length;
-        for( int i = 0; i < count; i++ )
+        for (int i = 0; i < count; i++)
         {
             Object arg = arguments[i];
-            if( i > 0 )
+            if (i > 0)
             {
                 buffer.append(", ");
             }
@@ -94,14 +94,19 @@ public class RecordedInvocation
         return buffer.toString();
     }
 
+//----------------------------------------------------------------------------------------------------------------------
+// Other Methods
+//----------------------------------------------------------------------------------------------------------------------
+
     /**
      * Add a string representation of <code>input</code> to <code>buffer</code>.
+     *
      * @param buffer
      * @param input
      */
-    protected void convert( StringBuffer buffer, Object input )
+    protected void convert(StringBuffer buffer, Object input)
     {
-        if( input == null )
+        if (input == null)
         {
             buffer.append("<null>");
             return;
@@ -109,7 +114,7 @@ public class RecordedInvocation
 
         // Primitive types, and non-object arrays
         // use toString().
-        if( !( input instanceof Object[] ) )
+        if (!(input instanceof Object[]))
         {
             buffer.append(input.toString());
             return;
@@ -119,11 +124,11 @@ public class RecordedInvocation
             buffer.append("(");
             buffer.append(ProxyUtils.getJavaClassName(input.getClass()));
             buffer.append("){");
-            Object[] array = ( Object[] ) input;
+            Object[] array = (Object[]) input;
             int count = array.length;
-            for( int i = 0; i < count; i++ )
+            for (int i = 0; i < count; i++)
             {
-                if( i > 0 )
+                if (i > 0)
                 {
                     buffer.append(", ");
                 }
@@ -134,4 +139,14 @@ public class RecordedInvocation
             buffer.append("}");
         }
     }
+
+    /**
+     * Get the arguments.
+     *
+     * @return Object[]
+     */
+    public Object[] getArguments()
+    {
+        return ArrayUtils.clone(arguments);
+    }
 }