Re: [io] ClosedOutputStream#flush

2017-08-15 Thread Elijah Zupancic
Please omit this line from the program: BufferedOutputStream bout = new BufferedOutputStream(fout, 9); On Tue, Aug 15, 2017 at 6:12 PM, Elijah Zupancic wrote: > I inspected the code path and I think this bug has merit. Consider the > following sample application:

Re: [io] ClosedOutputStream#flush

2017-08-15 Thread Elijah Zupancic
I inspected the code path and I think this bug has merit. Consider the following sample application: import org.apache.commons.io.IOUtils; import org.apache.commons.io.output.CloseShieldOutputStream; import java.io.BufferedOutputStream; import java.io.File; import java.io.FileInputStream; import

[io] ClosedOutputStream#flush

2017-08-10 Thread Tomas Celaya
Would anyone mind taking a look at this issue regarding the flush method on ClosedOutputStream? https://issues.apache.org/jira/browse/IO-546 The change is relatively trivial and the attached patch includes a test case. I understand the impact is significant but I think it would make