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

[Ales Justin]

btw: does this even have a JBVFS jira?

Ales Justin wrote:
> 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();
>> +
>>
>