You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2004/12/04 21:49:23 UTC

svn commit: r109823 - /geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java

Author: djencks
Date: Sat Dec  4 12:49:23 2004
New Revision: 109823

URL: http://svn.apache.org/viewcvs?view=rev&rev=109823
Log:
clarify field name
Modified:
   geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java

Modified: geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java
Url: http://svn.apache.org/viewcvs/geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java?view=diff&rev=109823&p1=geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java&r1=109822&p2=geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java&r2=109823
==============================================================================
--- geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java	(original)
+++ geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/util/URLPattern.java	Sat Dec  4 12:49:23 2004
@@ -24,11 +24,11 @@
 
 
 /**
- * Utility class for <code>WebModuleConfiguration</code>.  This class is used to generate qualified patterns, HTTP
+ * Utility class for <code>ModuleConfiguration</code>.  This class is used to generate qualified patterns, HTTP
  * method sets, complements of HTTP method sets, and HTTP method sets w/ transport restrictions for URL patterns that
  * are found in the web deployment descriptor.
  * @version $Rev$ $Date$
- * @see org.apache.geronimo.security.jacc.WebModuleConfiguration
+ * @see org.apache.geronimo.security.jacc.ModuleConfiguration
  */
 public class URLPattern {
     private final static String[] HTTP_METHODS = {"GET", "POST", "PUT", "DELETE", "HEAD", "OPTIONS", "TRACE"};
@@ -37,16 +37,16 @@
     private final static int INTEGRAL = 0x01;
     private final static int CONFIDENTIAL = 0x02;
 
-    private URLPatternCheck type;
-    private String pattern;
-    private int mask;
+    private final URLPatternCheck type;
+    private final String pattern;
+    private int httpMethodsMask;
     private int transport;
-    private HashSet roles = new HashSet();
+    private final HashSet roles = new HashSet();
 
     /**
      * Construct an instance of the utility class for <code>WebModuleConfiguration</code>.
      * @param pat the URL pattern that this instance is to collect information on
-     * @see org.apache.geronimo.security.jacc.WebModuleConfiguration
+     * @see org.apache.geronimo.security.jacc.ModuleConfiguration
      * @see "JSR 115, section 3.1.3" Translating Servlet Deployment Descriptors
      */
     public URLPattern(String pat) {
@@ -104,14 +104,14 @@
      */
     public void addMethod(String method) {
         if (method.length() == 0) {
-            mask = 0xFF;
+            httpMethodsMask = 0xFF;
             return;
         }
 
         boolean found = false;
         for (int j = 0; j < HTTP_METHODS.length; j++) {
             if (method.equals(HTTP_METHODS[j])) {
-                mask |= HTTP_MASKS[j];
+                httpMethodsMask |= HTTP_MASKS[j];
                 found = true;
 
                 break;
@@ -128,7 +128,7 @@
         StringBuffer buffer = null;
 
         for (int i = 0; i < HTTP_MASKS.length; i++) {
-            if ((mask & HTTP_MASKS[i]) > 0) {
+            if ((httpMethodsMask & HTTP_MASKS[i]) > 0) {
                 if (buffer == null) {
                     buffer = new StringBuffer();
                 } else {
@@ -145,7 +145,7 @@
         StringBuffer buffer = null;
 
         for (int i = 0; i < HTTP_MASKS.length; i++) {
-            if ((mask & HTTP_MASKS[i]) == 0) {
+            if ((httpMethodsMask & HTTP_MASKS[i]) == 0) {
                 if (buffer == null) {
                     buffer = new StringBuffer();
                 } else {