Author: mrglavas
Date: Thu Sep 29 01:04:18 2011
New Revision: 1177138

URL: http://svn.apache.org/viewvc?rev=1177138&view=rev
Log:
Eliminate redundant String.indexOf() calls.

Modified:
    xml/commons/trunk/java/src/org/apache/xml/resolver/helpers/Namespaces.java

Modified: 
xml/commons/trunk/java/src/org/apache/xml/resolver/helpers/Namespaces.java
URL: 
http://svn.apache.org/viewvc/xml/commons/trunk/java/src/org/apache/xml/resolver/helpers/Namespaces.java?rev=1177138&r1=1177137&r2=1177138&view=diff
==============================================================================
--- xml/commons/trunk/java/src/org/apache/xml/resolver/helpers/Namespaces.java 
(original)
+++ xml/commons/trunk/java/src/org/apache/xml/resolver/helpers/Namespaces.java 
Thu Sep 29 01:04:18 2011
@@ -45,9 +45,10 @@ public class Namespaces {
     public static String getPrefix(Element element) {
         String name = element.getTagName();
         String prefix = "";
-
-        if (name.indexOf(':') > 0) {
-            prefix = name.substring(0, name.indexOf(':'));
+        
+        final int indexOfColon = name.indexOf(':');
+        if (indexOfColon > 0) {
+            prefix = name.substring(0, indexOfColon);
         }
 
         return prefix;
@@ -62,9 +63,10 @@ public class Namespaces {
      */
     public static String getLocalName(Element element) {
         String name = element.getTagName();
-
-        if (name.indexOf(':') > 0) {
-            name = name.substring(name.indexOf(':')+1);
+        
+        final int indexOfColon = name.indexOf(':');
+        if (indexOfColon > 0) {
+            name = name.substring(indexOfColon + 1);
         }
 
         return name;



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to