[Jboss-svn-commits] JBL Code SVN: r1507 - trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite

[<jbo...@li...>]

Author: rem...@jb...
Date: 2005-11-04 06:52:49 -0500 (Fri, 04 Nov 2005)
New Revision: 1507

Modified:
   trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java
Log:
- The cost of creating the TomcatResolver instance is likely non existent
  (and the rest of the algorithm is not GC friendly at this point, so ...).

Modified: trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java
===================================================================
--- trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java	2005-11-04 10:07:06 UTC (rev 1506)
+++ trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java	2005-11-04 11:52:49 UTC (rev 1507)
@@ -62,13 +62,6 @@
     
     
     /**
-     * Thread local Tomcat resolver.
-     * Note: this could also go to a request note. 
-     */
-    protected ThreadLocal resolver = new ThreadLocal();
-    
-    
-    /**
      * If rewriting occurs, the whole request will be processed again.
      */
     protected ThreadLocal invoked = new ThreadLocal();
@@ -239,11 +232,7 @@
             return;
         }
         
-        Resolver resolver = (Resolver) this.resolver.get();
-        if (resolver == null) {
-            resolver = new TomcatResolver(request);
-            this.resolver.set(resolver);
-        }
+        TomcatResolver resolver = new TomcatResolver(request);
         
         invoked.set(Boolean.TRUE);