boris-petrov opened a new pull request #151: URL: https://github.com/apache/commons-vfs/pull/151
I'm not sure whether changing the visibility of `AbstractFileObject.getOutputStream` and marking it as `final` is not a breaking change. This change is not *strictly* needed but I did it to protect from someone accidentally calling it from the outside like was done in `RamFileSystem`. `AbstractFileObject.getOutputStream` is an "internal" method which users should not call because closing the stream returned from it doesn't call the needed hooks - like `endOutput`. That was why `RamFileOutputStream.close` did that explicitly (which is not nice because when creating a new file in the RAM provider, `endOutput` would be called twice). See also #150. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
