You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by mr...@apache.org on 2011/12/28 23:54:01 UTC

svn commit: r1225369 - in /xalan/java/trunk/src/org/apache/xalan/xsltc/dom: DocumentCache.java LoadDocument.java

Author: mrglavas
Date: Wed Dec 28 22:54:01 2011
New Revision: 1225369

URL: http://svn.apache.org/viewvc?rev=1225369&view=rev
Log:
Minor performance improvement. String.length() == 0 tends to be slightly faster than String.equals("").

Modified:
    xalan/java/trunk/src/org/apache/xalan/xsltc/dom/DocumentCache.java
    xalan/java/trunk/src/org/apache/xalan/xsltc/dom/LoadDocument.java

Modified: xalan/java/trunk/src/org/apache/xalan/xsltc/dom/DocumentCache.java
URL: http://svn.apache.org/viewvc/xalan/java/trunk/src/org/apache/xalan/xsltc/dom/DocumentCache.java?rev=1225369&r1=1225368&r2=1225369&view=diff
==============================================================================
--- xalan/java/trunk/src/org/apache/xalan/xsltc/dom/DocumentCache.java (original)
+++ xalan/java/trunk/src/org/apache/xalan/xsltc/dom/DocumentCache.java Wed Dec 28 22:54:01 2011
@@ -257,7 +257,7 @@ public final class DocumentCache impleme
 	CachedDocument doc;
 
     String uri = href;
-    if (baseURI != null && !baseURI.equals("")) {
+    if (baseURI != null && baseURI.length() != 0) {
         try {
             uri = SystemIDResolver.getAbsoluteURI(uri, baseURI);
         } catch (TransformerException te) {

Modified: xalan/java/trunk/src/org/apache/xalan/xsltc/dom/LoadDocument.java
URL: http://svn.apache.org/viewvc/xalan/java/trunk/src/org/apache/xalan/xsltc/dom/LoadDocument.java?rev=1225369&r1=1225368&r2=1225369&view=diff
==============================================================================
--- xalan/java/trunk/src/org/apache/xalan/xsltc/dom/LoadDocument.java (original)
+++ xalan/java/trunk/src/org/apache/xalan/xsltc/dom/LoadDocument.java Wed Dec 28 22:54:01 2011
@@ -158,13 +158,13 @@ public final class LoadDocument {
         MultiDOM multiplexer = (MultiDOM)dom;
 
         // Prepend URI base to URI (from context)
-        if (base != null && !base.equals("")) {
+        if (base != null && base.length() != 0) {
             uri = SystemIDResolver.getAbsoluteURI(uri, base);
         }
 
         // Return an empty iterator if the URI is clearly invalid
         // (to prevent some unncessary MalformedURL exceptions).
-        if (uri == null || uri.equals("")) {
+        if (uri == null || uri.length() == 0) {
             return(EmptyIterator.getInstance());
         }
         



---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-cvs-help@xml.apache.org