[jboss-cvs] JBossAS SVN: r87396 - in projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins: vfs/helpers and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: alesj
Date: 2009-04-15 18:24:49 -0400 (Wed, 15 Apr 2009)
New Revision: 87396

Modified:
   projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipEntryContext.java
   projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/vfs/helpers/PathTokenizer.java
Log:
Put fields to the top.

Modified: projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipEntryContext.java
===================================================================
--- projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipEntryContext.java	2009-04-15 22:18:59 UTC (rev 87395)
+++ projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipEntryContext.java	2009-04-15 22:24:49 UTC (rev 87396)
@@ -94,6 +94,7 @@
  *
  * @author <a href="strukelj at parsek.net">Marko Strukelj</a>
  * @author <a href="ales.justin at jboss.org">Ales Justin</a>
+ * @author <a href="david.lloyd at jboss.com">David M. Lloyd</a>
  * @version $Revision: 1.0 $
  */
 public class ZipEntryContext extends AbstractVFSContext
@@ -114,6 +115,9 @@
          log.info("VFS force nested jars copy-mode is enabled.");
    }
 
+   /** The empty bytes const */
+   private static final byte[] NO_BYTES = new byte[0];
+
    /** Abstracted access to zip archive - either ZipFileWrapper or ZipStreamWrapper */
    private ZipWrapper zipSource;
 
@@ -1029,8 +1033,6 @@
       return false;
    }
 
-   private static final byte[] NO_BYTES = new byte[0];
-
    /**
     * Contents of the file represented by a given handler
     *

Modified: projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/vfs/helpers/PathTokenizer.java
===================================================================
--- projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/vfs/helpers/PathTokenizer.java	2009-04-15 22:18:59 UTC (rev 87395)
+++ projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/vfs/helpers/PathTokenizer.java	2009-04-15 22:24:49 UTC (rev 87396)
@@ -31,6 +31,7 @@
  * 
  * @author <a href="adrian at jboss.com">Adrian Brock</a>
  * @author <a href="ales.justin at jboss.com">Ales Justin</a>
+ * @author <a href="david.lloyd at jboss.com">David M. Lloyd</a>
  * @version $Revision: 1.1 $
  */
 @SuppressWarnings({"StringEquality"})
@@ -48,6 +49,12 @@
    /** Flag permission */
    private static Permission flagPermission = new RuntimePermission(PathTokenizer.class.getName() + ".setErrorOnSuspiciousTokens");
 
+   /** Token states */
+   private static final int STATE_INITIAL = 0;
+   private static final int STATE_NORMAL = 1;
+   private static final int STATE_MAYBE_CURRENT_PATH = 2;
+   private static final int STATE_MAYBE_REVERSE_PATH = 3;
+
    /**
     * Utility class
     */
@@ -84,11 +91,6 @@
       return buffer.toString();
    }
 
-   private static final int STATE_INITIAL = 0;
-   private static final int STATE_NORMAL = 1;
-   private static final int STATE_MAYBE_CURRENT_PATH = 2;
-   private static final int STATE_MAYBE_REVERSE_PATH = 3;
-
    /**
     * Get the tokens that comprise this path.
     * 
@@ -174,7 +176,6 @@
                         throw new IllegalArgumentException("Illegal suspicious token in path: " + path);
                      }
                      state = STATE_NORMAL;
-                     continue;
                   }
                }
             }