You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by jc...@apache.org on 2005/08/26 14:18:07 UTC

svn commit: r240216 - /jakarta/commons/sandbox/proxy/trunk/src/java/org/apache/commons/proxy/factory/javassist/JavassistProxyFactory.java

Author: jcarman
Date: Fri Aug 26 05:18:06 2005
New Revision: 240216

URL: http://svn.apache.org/viewcvs?rev=240216&view=rev
Log:
Now uses log variable from parent class.

Modified:
    jakarta/commons/sandbox/proxy/trunk/src/java/org/apache/commons/proxy/factory/javassist/JavassistProxyFactory.java

Modified: jakarta/commons/sandbox/proxy/trunk/src/java/org/apache/commons/proxy/factory/javassist/JavassistProxyFactory.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/proxy/trunk/src/java/org/apache/commons/proxy/factory/javassist/JavassistProxyFactory.java?rev=240216&r1=240215&r2=240216&view=diff
==============================================================================
--- jakarta/commons/sandbox/proxy/trunk/src/java/org/apache/commons/proxy/factory/javassist/JavassistProxyFactory.java (original)
+++ jakarta/commons/sandbox/proxy/trunk/src/java/org/apache/commons/proxy/factory/javassist/JavassistProxyFactory.java Fri Aug 26 05:18:06 2005
@@ -24,8 +24,6 @@
 import javassist.NotFoundException;
 import org.aopalliance.intercept.MethodInterceptor;
 import org.aopalliance.intercept.MethodInvocation;
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
 import org.apache.commons.proxy.ObjectProvider;
 import org.apache.commons.proxy.exception.ObjectProviderException;
 import org.apache.commons.proxy.exception.ProxyFactoryException;
@@ -42,7 +40,6 @@
  */
 public class JavassistProxyFactory extends AbstractProxyFactory
 {
-    private static final Log log = LogFactory.getLog( JavassistProxyFactory.class );
     private static int classNumber = 0;
     private static final ClassPool classPool = ClassPool.getDefault();
 
@@ -121,7 +118,7 @@
             constructorBody.append( "\tthis.target = $1;\n" );
             for( int i = 0; i < argumentTypes.length; i++ )
             {
-                constructorBody.append( "\tthis.argument" + i + " = $" + ( 2 + i ) + ";\n" );
+                constructorBody.append( "\tthis.argument" ).append( i ).append( " = $" ).append( 2 + i ).append( ";\n" );
 
             }
             constructorBody.append( "}" );
@@ -153,10 +150,10 @@
         {
             proceedBody.append( "\t" );
         }
-        proceedBody.append( "target." + method.getName() + "(" );
+        proceedBody.append( "target." ).append( method.getName() ).append( "(" );
         for( int i = 0; i < argumentTypes.length; ++i )
         {
-            proceedBody.append( "argument" + i );
+            proceedBody.append( "argument" ).append( i );
             if( i != argumentTypes.length - 1 )
             {
                 proceedBody.append( ", " );



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org