[Sblim-issues] [ sblim-Bugs-3023340 ] CIMObjectFactory uses # constructor instead of valueOf

[SourceForge.net <no...@so...>]

Bugs item #3023340, was opened at 2010-06-30 07:37
Message generated for change (Tracker Item Submitted) made by blaschke-oss
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=712784&aid=3023340&group_id=128809

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Java Client (JSR48)
Group: Performance
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Dave Blaschke (blaschke-oss)
Assigned to: Dave Blaschke (blaschke-oss)
Summary: CIMObjectFactory uses # constructor instead of valueOf

Initial Comment:
CIMObjectFactory's char16 ValueFactory method calls the Character constructor instead of using the better performing valueOf() method.

Bug: Method org.sblim.cimclient.internal.cimxml.sax.CIMObjectFactory$11.make(String) invokes inefficient new Character(char) constructor; use Character.valueOf(char) instead
Pattern id: DM_NUMBER_CTOR, type: Bx, category: PERFORMANCE

Using new Integer(int) is guaranteed to always result in a new object whereas Integer.valueOf(int) allows caching of values to be done by the compiler, class library, or JVM. Using of cached values avoids object allocation and the code will be faster. 


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=712784&aid=3023340&group_id=128809