[
http://issues.apache.org/jira/browse/XALANJ-2217?page=comments#action_12332734
]
Brian Minchau commented on XALANJ-2217:
---------------------------------------
Dave, I'm ready to commit the patch, but have one issue I'm not sure of.
-----------------------------------------------
Most are trivial .toString() methods called on a String, object, so the call is
useless, and could even
create an additional String for no reason at all. I'm fine with these.
------------------------------------------------
OpMap has this change
new Integer(stepType).toString()
-------------> String.valueOf(stepType)
where stepType is an int. I looked up the Javadoc on String.valueOf(int) at
http://java.sun.com/j2se/1.4.2/docs/api/java/lang/String.html#valueOf(int)
and it said this:
"The representation is exactly the one returned by the Integer.toString
method of one argument. "
so I'm OK with that one.
------------------------------------------------
The only change in the whole patch that has me concerned is the one in
ElemNumber
(new Character(table.getChar((int)val - 1))).toString();
-----------> String.valueOf(table.getChar((int)val - 1));
table.getChar(int index) returns a char. I'm not sure that for a given char,
ch, that
String.valueOf(ch)
is the same as
(new Character(ch)).toString();
The javadoc at:
http://java.sun.com/j2se/1.4.2/docs/api/java/lang/String.html#valueOf(char)
says this:
>> Returns the string representation of the char argument.
The javadoc for Charcter.toString() at:
http://java.sun.com/j2se/1.4.2/docs/api/java/lang/Character.html#toString()
says this:
>> Returns a String object representing this Character's value.
>>The result is a string of length 1 whose sole component is the primitive
>> char value represented by this Character object.
So are these always the same? I'm worried about some subtle differences between
char and Character.
Perhaps the current locale can leak in somehow? I don't know for sure, just
have my worries about this one.
These are always the same, then I'm ready to commit the patch to the code base.
- Brian
> [PATCH] cleaup some String usage sillyness
> ------------------------------------------
>
> Key: XALANJ-2217
> URL: http://issues.apache.org/jira/browse/XALANJ-2217
> Project: XalanJ2
> Type: Improvement
> Versions: Latest Development Code
> Environment: n/a
> Reporter: Dave Brosius
> Priority: Trivial
> Attachments: silly_string_stuff.diff
>
> This patch cleans up some String class sillyness like creating an object just
> to convert to string, or calling toString on a String, etc.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
