[jboss-cvs] JBossAS SVN: r74500 - projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip.

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

Author: alesj
Date: 2008-06-12 15:41:37 -0400 (Thu, 12 Jun 2008)
New Revision: 74500

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/ZipFileWrapper.java
   projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipWrapper.java
Log:
javadocs

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	2008-06-12 19:29:55 UTC (rev 74499)
+++ projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipEntryContext.java	2008-06-12 19:41:37 UTC (rev 74500)
@@ -266,12 +266,11 @@
       // then we order these by name and only then we process them
       // this way we ensure that parent entries are processed before child entries
 
-      HashMap<String, ZipEntry> relevant = new HashMap<String, ZipEntry>();
+      Map<String, ZipEntry> relevant = new HashMap<String, ZipEntry>();
       zipSource.acquire();
       try
       {
          Enumeration<? extends ZipEntry> zipEntries = zipSource.entries();
-
          // zoom-in on entries under rootEntryPath - ignoring the rest
          while(zipEntries.hasMoreElements())
          {
@@ -282,8 +281,7 @@
             }
          }
 
-         TreeMap<String, ZipEntry> orderedRelevant = new TreeMap<String, ZipEntry>(relevant);
-
+         Map<String, ZipEntry> orderedRelevant = new TreeMap<String, ZipEntry>(relevant);
          for(Map.Entry<String, ZipEntry> entry : orderedRelevant.entrySet())
          {
             ZipEntry ent = entry.getValue();
@@ -340,7 +338,6 @@
                ZipEntryHandler wrapper = new ZipEntryHandler(this, parent, name, ent.isDirectory() == false);
                entries.put(wrapper.getLocalPathName(), new EntryInfo(wrapper, ent));
             }
-
          }
       }
       finally

Modified: projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipFileWrapper.java
===================================================================
--- projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipFileWrapper.java	2008-06-12 19:29:55 UTC (rev 74499)
+++ projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipFileWrapper.java	2008-06-12 19:41:37 UTC (rev 74500)
@@ -56,7 +56,6 @@
 
       if (forceNoReaper)
          log.info("VFS forced no-reaper-mode is enabled.");
-
    }
 
    private File file;
@@ -83,7 +82,7 @@
    {
       this.noReaperOverride = noReaperOverride;
       File rootFile = new File(rootPathURI);
-      if(!rootFile.isFile())
+      if(rootFile.isFile() == false)
          throw new RuntimeException("File not found: " + rootFile);
 
       init(rootFile, autoClean);

Modified: projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipWrapper.java
===================================================================
--- projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipWrapper.java	2008-06-12 19:29:55 UTC (rev 74499)
+++ projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/zip/ZipWrapper.java	2008-06-12 19:41:37 UTC (rev 74500)
@@ -33,10 +33,8 @@
  * @author <a href="strukelj at parsek.net">Marko Strukelj</a>
  * @version $Revision: 1.0 $
  */
-
 abstract class ZipWrapper
 {
-
    /** last known modifyTime of the zip source */
    protected long lastModified;
 
@@ -50,10 +48,10 @@
    /** number of streams currently open on this wrapper */
    private int refCount = 0;
 
-
-
    /**
-    * Returns true if underlying source's lastModified time has changed since previous call
+    * Returns true if underlying source's lastModified time has changed since previous call.
+    *
+    * @return true if modified, false othwewise
     */
    boolean hasBeenModified()
    {
@@ -93,7 +91,6 @@
       lastUsed = System.currentTimeMillis();
    }
 
-
    synchronized void release()
    {
       refCount--;