[jbossws-issues] [JBoss JIRA] Updated: (JBWS-2404) Efficient string handling
Anthony Whitford (JIRA)
jira-events at lists.jboss.org
Mon Nov 24 02:30:36 EST 2008
[ https://jira.jboss.org/jira/browse/JBWS-2404?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Anthony Whitford updated JBWS-2404:
-----------------------------------
Attachment: JBWS-2404.patch
Attached is a patch for a handful of issues (and a couple of bugs).
> Efficient string handling
> -------------------------
>
> Key: JBWS-2404
> URL: https://jira.jboss.org/jira/browse/JBWS-2404
> Project: JBoss Web Services
> Issue Type: Patch
> Security Level: Public(Everyone can see)
> Components: jbossws-native
> Affects Versions: jbossws-native-3.0.4
> Reporter: Anthony Whitford
> Attachments: JBWS-2404.patch
>
>
> Browsing the code, I came across some low hanging fruit around string usage. For example:
> native\modules\core\src\main\java\org\jboss\ws\metadata\wsdl\WSDLUtils.java 445 Use equals() to compare strings instead of '==' or '!='
> native\modules\core\src\main\java\org\jboss\ws\metadata\wsdl\WSDLUtils.java 549 String.indexOf(char) is faster than String.indexOf(String).
> native\modules\core\src\main\java\org\jboss\ws\metadata\wsdl\WSDLUtils.java 648 Use equals() to compare strings instead of '==' or '!='
> native\modules\core\src\main\java\org\jboss\ws\metadata\wsdl\WSDLUtils.java 664 String.indexOf(char) is faster than String.indexOf(String).
> Also noticed that there was the odd "bug", like:
> public String firstLetterUpperCase(String fname)
> {
> if (fname == "" || fname == null)
> throw new WSException("String passed is null");
> ...
> The null check should be first, and comparing to an empty string literal isn't exactly the same thing as detecting an empty string.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the jbossws-issues
mailing list