Not just boxing, but even unnecessary array creation.  Since tracef does have 
an overloaded signature of tracef(String fmt, Object… params)

So please do consider if (trace).  :)

On 22 Feb 2012, at 19:16, Sanne Grinovero wrote:

> It's easy to be fooled by the nice looking tracef API; can you see
> what's wrong with the next method?
> 
> log.tracef("%s: Committing view %s", cacheName, viewId);
> 
> 
> [solution below]
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> The signature for such a tracef method is
> tracef(String format, Object param1, Object param2);
> so integers, booleans, etcetera.. will be boxed into objects.
> _______________________________________________
> infinispan-dev mailing list
> [email protected]
> https://lists.jboss.org/mailman/listinfo/infinispan-dev

--
Manik Surtani
[email protected]
twitter.com/maniksurtani

Lead, Infinispan
http://www.infinispan.org




_______________________________________________
infinispan-dev mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/infinispan-dev

Reply via email to