Nice one! I will try it now! On Mon, Sep 19, 2011 at 10:07 AM, Geoffrey De Smet < ge0ffrey.spam(a)gmail.com&gt; wrote: > Hi guys, > > It's very easy to run findbugs on your module: > > cd droolsjbpm/drools-core > mvn site -DskipTests > firefox target/site/findbugs.html > > Most of the "bugs" on that webpage are ignorable, but there are a couple > of very interesting cases. > Just look for category PERFORMANCE. > > > For example in drools-core: > Method new > org.drools.io.impl.ResourceChangeScannerConfigurationImpl(Properties) > makes inefficient use of keySet iterator instead of entrySet iterator > > This code: > public ResourceChangeScannerConfigurationImpl(Properties properties) { > for( Object key : properties.keySet() ) { > setProperty( (String) key, (String) properties.get( key ) ); > } > } > should be the faster alternative: > public ResourceChangeScannerConfigurationImpl(Properties properties) { > for(Map.Entry<Object, Object> entry : properties.entrySet() ) { > setProperty( (String) entry.getKey(), (String) > entry.getValue()); > } > } > > -- > With kind regards, > Geoffrey De Smet > > > _______________________________________________ > rules-dev mailing list > rules-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/rules-dev > -- - CTO @ http://www.plugtree.com - MyJourney @ http://salaboy.wordpress.com - Co-Founder @ http://www.jugargentina.org - Co-Founder @ http://www.jbug.com.ar - Salatino "Salaboy" Mauricio - _______________________________________________ rules-dev mailing list rules-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-dev