[jboss-cvs] jboss-cvs-commits Digest, Vol 38, Issue 193

Ales Justin ales.justin at gmail.com
Sun Aug 16 07:31:32 EDT 2009


ZipReleaseAfterCopyTestCase also looks broken - VFSUtils::temp doesn't 
create zipFile at all, hence the test will always pass, w/ or w/o 
::releaseHandles.

Next bug is that DelegateHandler doesn't delegate ::releaseHandles.

Both things are now fixed.

Ales Justin wrote:
> All I see is a no-op impl of this. Forgot to commit something?
> (e.g. override in ZipEntryHandler --> 
> ZipEntryContext::releaseHandles(ZipEntryHandler))
> 
> ---------- Forwarded message ----------
> From: ** <jboss-cvs-commits-request at lists.jboss.org 
> <mailto:jboss-cvs-commits-request at lists.jboss.org>>
> Date: Fri, Aug 14, 2009 at 9:51 AM
> Subject: jboss-cvs-commits Digest, Vol 38, Issue 193
> To: jboss-cvs-commits at lists.jboss.org 
> <mailto:jboss-cvs-commits at lists.jboss.org>
> 
> 
> Modified: 
> projects/vfs/branches/Branch_2_1/src/main/java/org/jboss/virtual/spi/VirtualFileHandler.java
> ===================================================================
> --- 
> projects/vfs/branches/Branch_2_1/src/main/java/org/jboss/virtual/spi/VirtualFileHandler.java 
>        2009-08-14 06:41:59 UTC (rev 92337)
> +++ 
> projects/vfs/branches/Branch_2_1/src/main/java/org/jboss/virtual/spi/VirtualFileHandler.java 
>        2009-08-14 06:49:42 UTC (rev 92338)
> @@ -34,7 +34,7 @@
> 
>    /**
> +    * Temporarily release file handles
> +    */
> +   void releaseHandles();
> +
> 



More information about the jboss-cvs-commits mailing list