You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/12/07 23:51:22 UTC
svn commit: r1211687 -
/commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/ObjectCreateRule.java
Author: simonetripodi
Date: Wed Dec 7 22:51:22 2011
New Revision: 1211687
URL: http://svn.apache.org/viewvc?rev=1211687&view=rev
Log:
format violations reported by checkstyle
Modified:
commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/ObjectCreateRule.java
Modified: commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/ObjectCreateRule.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/ObjectCreateRule.java?rev=1211687&r1=1211686&r2=1211687&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/ObjectCreateRule.java (original)
+++ commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/ObjectCreateRule.java Wed Dec 7 22:51:22 2011
@@ -66,7 +66,8 @@ public class ObjectCreateRule
{
invocations.add( new RecordedInvocation( method, args ) );
}
- if ( hasDelegate ) {
+ if ( hasDelegate )
+ {
return proxy.invoke( delegate, args );
}
return proxy.invokeSuper( obj, args );
@@ -99,7 +100,7 @@ public class ObjectCreateRule
ProxyManager( Class<?> clazz, Constructor<?> constructor, Object[] constructorArguments, Digester digester )
{
this.clazz = clazz;
- hasDefaultConstructor = getAccessibleConstructor(clazz, new Class[0]) != null;
+ hasDefaultConstructor = getAccessibleConstructor( clazz, new Class[0] ) != null;
this.constructor = constructor;
Class<?>[] argTypes = constructor.getParameterTypes();
templateConstructorArguments = new Object[argTypes.length];
@@ -107,14 +108,14 @@ public class ObjectCreateRule
{
for ( int i = 0; i < templateConstructorArguments.length; i++ )
{
- if ( argTypes[i].equals(boolean.class) )
+ if ( argTypes[i].equals( boolean.class ) )
{
templateConstructorArguments[i] = Boolean.FALSE;
continue;
}
if ( argTypes[i].isPrimitive() )
{
- templateConstructorArguments[i] = convert("0", argTypes[i]);
+ templateConstructorArguments[i] = convert( "0", argTypes[i] );
continue;
}
templateConstructorArguments[i] = null;
@@ -140,7 +141,8 @@ public class ObjectCreateRule
arraycopy( templateConstructorArguments, 0, constructorArguments, 0, constructorArguments.length );
digester.pushParams( constructorArguments );
- DeferredConstructionCallback callback = new DeferredConstructionCallback(constructor, constructorArguments);
+ DeferredConstructionCallback callback =
+ new DeferredConstructionCallback( constructor, constructorArguments );
Object result;
@@ -150,7 +152,7 @@ public class ObjectCreateRule
enhancer.setSuperclass( clazz );
enhancer.setCallback( callback );
enhancer.setClassLoader( digester.getClassLoader() );
- enhancer.setInterceptDuringConstruction(false);
+ enhancer.setInterceptDuringConstruction( false );
if ( hasDefaultConstructor )
{
result = enhancer.create();
@@ -175,10 +177,11 @@ public class ObjectCreateRule
return result;
}
- void finalize( Object proxy ) throws Exception
+ void finalize( Object proxy )
+ throws Exception
{
digester.popParams();
- ( ( DeferredConstructionCallback ) ( (Factory) proxy).getCallback( 0 ) ).establishDelegate();
+ ( (DeferredConstructionCallback) ( (Factory) proxy ).getCallback( 0 ) ).establishDelegate();
}
}
@@ -359,7 +362,7 @@ public class ObjectCreateRule
clazz.getName(),
Arrays.toString( constructorArgumentTypes ) ) );
}
- proxyManager = new ProxyManager(clazz, constructor, defaultConstructorArguments, getDigester());
+ proxyManager = new ProxyManager( clazz, constructor, defaultConstructorArguments, getDigester() );
}
instance = proxyManager.createProxy();
}
@@ -377,7 +380,7 @@ public class ObjectCreateRule
if ( proxyManager != null )
{
- proxyManager.finalize(top);
+ proxyManager.finalize( top );
}
if ( getDigester().getLogger().isDebugEnabled() )