boris-petrov opened a new pull request #166:
URL: https://github.com/apache/commons-vfs/pull/166


   This also fixes inability to close a stream from a different thread than the 
one that opened it. See discussion in 
https://github.com/apache/commons-vfs/pull/155.
   
   I tried to mimic pretty much exactly the code before (to minimize the 
changes) but to fix the underlying issue. There are still problems like 
concurrently closing one stream (which potentially does 
`this.fileContentThreadData = null` if it was the last stream) and opening a 
new one (which might add a stream to the existing `this.fileContentThreadData`, 
causing it to be "lost" because of the setting of the variable to `null` during 
the closing) but 1) they were also there before; 2) they will be very rare and 
3) are the subject of a later PR.
   
   cc @garydgregory, @MaxKellermann


----------------------------------------------------------------
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]


Reply via email to