You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2012/02/13 17:43:54 UTC

svn commit: r1243599 - /myfaces/core/trunk/shared/src/main/java/org/apache/myfaces/shared/resource/ResourceImpl.java

Author: lu4242
Date: Mon Feb 13 16:43:54 2012
New Revision: 1243599

URL: http://svn.apache.org/viewvc?rev=1243599&view=rev
Log:
MYFACES-3458 [perf] Store URL at class ResourceImpl

Modified:
    myfaces/core/trunk/shared/src/main/java/org/apache/myfaces/shared/resource/ResourceImpl.java

Modified: myfaces/core/trunk/shared/src/main/java/org/apache/myfaces/shared/resource/ResourceImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/shared/src/main/java/org/apache/myfaces/shared/resource/ResourceImpl.java?rev=1243599&r1=1243598&r2=1243599&view=diff
==============================================================================
--- myfaces/core/trunk/shared/src/main/java/org/apache/myfaces/shared/resource/ResourceImpl.java (original)
+++ myfaces/core/trunk/shared/src/main/java/org/apache/myfaces/shared/resource/ResourceImpl.java Mon Feb 13 16:43:54 2012
@@ -44,6 +44,7 @@ public class ResourceImpl extends Resour
     private ResourceMeta _resourceMeta;
     private ResourceLoader _resourceLoader;
     private ResourceHandlerSupport _resourceHandlerSupport;
+    private URL _url; 
     
     public ResourceImpl(ResourceMeta resourceMeta, 
             ResourceLoader resourceLoader, ResourceHandlerSupport support, String contentType)
@@ -195,7 +196,13 @@ public class ResourceImpl extends Resour
     @Override
     public URL getURL()
     {
-        return getResourceLoader().getResourceURL(_resourceMeta);
+        // For the default algorithm, it is safe to assume the resource
+        // URL will not change over resource lifetime. See MYFACES-3458
+        if (_url == null)
+        {
+            _url = getResourceLoader().getResourceURL(_resourceMeta);
+        }
+        return _url;
     }
 
     @Override