Bug 44366

Summary: [patch] POIFSFileSystem does not close input stream
Product: POI Reporter: Josh Micich <josh>
Component: POIFSAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 3.0-dev   
Target Milestone: ---   
Hardware: Other   
OS: other   
Attachments: svn diff showing changes to 6 files
tar bz2 of 9 files (1 test case and 8 test suites)
svn diff of one modified file: POIFSFileSystem.java
tar bz2 of one added test case

Description Josh Micich 2008-02-05 23:44:02 UTC
Added try/finally code to POIFSFileSystem constructor to close input stream. 
Added junit.

Added comment to HSSFWorkbook to make this behaviour clear.

Added full hierarchy of test suites to aggregate all POI tests.

Fixed excluded junit TestEmptyDocument, and added to suite.

Updated build.xml to include TestEmptyDocument and mask out new Test Suite 
classes (naming convention: All*Tests.java).
Comment 1 Josh Micich 2008-02-05 23:46:10 UTC
Created attachment 21480 [details]
svn diff showing changes to 6 files
Comment 2 Josh Micich 2008-02-05 23:47:32 UTC
Created attachment 21481 [details]
tar bz2 of 9 files (1 test case and 8 test suites)
Comment 3 Nick Burch 2008-02-06 03:41:40 UTC
I think we'll want to discuss this one on dev@ first

Can you do a post about why you think this change should be made, what affects
it'll have on existing code etc? We can then decide about going ahead with it.
Comment 4 Josh Micich 2008-02-15 18:46:58 UTC
Created attachment 21543 [details]
svn diff of one modified file: POIFSFileSystem.java
Comment 5 Josh Micich 2008-02-15 18:47:32 UTC
Created attachment 21544 [details]
tar bz2 of one added test case
Comment 6 Nick Burch 2008-02-21 07:36:08 UTC
Thanks for this updated patch Josh. Now applied to svn trunk