<AWT Dev> Review Request: JDK-8027992 FileInputStream and BufferedInputStream should be closed in sun.applet.*

Anthony Petrov anthony.petrov at oracle.com
Mon Nov 18 06:56:20 PST 2013


Still looks fine.

--
best regards,
Anthony

On 11/18/2013 10:58 AM, Petr Pchelko wrote:
> Hello,
>
> Please have a look on an updated version of the fix:
> http://cr.openjdk.java.net/~pchelko/8027992/webrev.01/
>
> According to Sergey's offline comment I've use a better pattern for try-with-resources with wrapped streams.
>
> With best regards. Petr.
>
> On 13.11.2013, at 20:51, Anthony Petrov <anthony.petrov at oracle.com> wrote:
>
>> Looks fine to me.
>>
>> --
>> best regards,
>> Anthony
>>
>> On 11/13/2013 12:13 PM, Petr Pchelko wrote:
>>> Hello,
>>>
>>> Please review the fix for the issue: https://bugs.openjdk.java.net/browse/JDK-8027992
>>> The fix is available at: http://cr.openjdk.java.net/~pchelko/8027992/webrev/
>>>
>>> A simple fix, try-with-resources blocks added when needed.
>>>
>>> Thank you.
>>> With best regards. Petr.
>>>
>


More information about the awt-dev mailing list