Good!!

> @@ -289,7 +290,7 @@ public class OnamiRunner
>              {
>                  LOGGER.finer( "  Start introspecting class: " + 
> clazz.getName() );
>              }
> -            final List<Module> allModules = new ArrayList<Module>( 1 );
> +            final List<Module> allModules = new ArrayList<Module>();
>
>              // Setup the handlers
>              final GuiceProvidedModuleHandler guiceProvidedModuleHandler = 
> new GuiceProvidedModuleHandler();
>
> Modified: 
> incubator/onami/trunk/test/src/main/java/org/apache/onami/test/handler/GuiceModuleHandler.java
> URL: 
> http://svn.apache.org/viewvc/incubator/onami/trunk/test/src/main/java/org/apache/onami/test/handler/GuiceModuleHandler.java?rev=1435848&r1=1435847&r2=1435848&view=diff
> ==============================================================================
> --- 
> incubator/onami/trunk/test/src/main/java/org/apache/onami/test/handler/GuiceModuleHandler.java
>  (original)
> +++ 
> incubator/onami/trunk/test/src/main/java/org/apache/onami/test/handler/GuiceModuleHandler.java
>  Sun Jan 20 14:28:25 2013
> @@ -41,7 +41,7 @@ public final class GuiceModuleHandler
>
>      private static final Logger LOGGER = Logger.getLogger( 
> GuiceModuleHandler.class.getName() );
>
> -    private final List<Module> modules = new ArrayList<Module>( 1 );
> +    private final List<Module> modules = new ArrayList<Module>();

I think you can use LinkedList here as well - WDYT?
TIA!
-Simo

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/

Reply via email to