commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Henrib <hbies...@gmail.com>
Subject Re: svn commit: r801035 - in /commons/proper/jexl/branches/2.0/src/main/java/org/apache/commons/jexl: Interpreter.java JexlArithmetic.java
Date Wed, 05 Aug 2009 00:19:13 GMT



sebb-3 wrote:
> 
> Author: sebb
> Date: Wed Aug  5 00:08:11 2009
> New Revision: 801035
> 
> URL: http://svn.apache.org/viewvc?rev=801035&view=rev
> Log:
> Make boxing explicit (use Integer.valueOf() for numbers that are likely to
> be small)
> Remove Long=>long conversions where the return value is Long anyway!
> 
> Modified:
>    
> commons/proper/jexl/branches/2.0/src/main/java/org/apache/commons/jexl/Interpreter.java
>    
> commons/proper/jexl/branches/2.0/src/main/java/org/apache/commons/jexl/JexlArithmetic.java
> 
> 

The findbugs (maven plugin version) was flagging usage of new Long(...) as a
performance warning so I changed them to autoboxing which it seemed happy
with sometime ago.
Not sure if these have really an impact...

-- 
View this message in context: http://www.nabble.com/svn-commit%3A-r801035---in--commons-proper-jexl-branches-2.0-src-main-java-org-apache-commons-jexl%3A-Interpreter.java-JexlArithmetic.java-tp24818432p24818509.html
Sent from the Commons - Dev mailing list archive at Nabble.com.


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


Mime
View raw message