It looks as if the new proposed fix will mean cached data isn't flushed
before you close
the stream ??
-phil.
On 6/14/12 1:25 PM, Martin Desruisseaux wrote:
Le 14/06/12 19:37, Phil Race a écrit :
I think it acceptable to make it call checkClosed() conpatibility
wise since apps must already be ready
for an IOException and are actually less like to be ready for the NPE ..
After a second look to the code, it appears to have been the original
intend. All ImageInputStream and ImageOutputStream implementations
invoke super.close(), which in turn invoke checkClosed(). It just
appears that FileCacheImageOutputStream is making this invocation too
late - we get a NPE before to reach (indirectly) the checkClosed() call.
I attached to this email the new proposed fix.
Martin
