Hi,

On Fri, 2005-09-16 at 13:15 -0600, Tom Tromey wrote:
> from  Tom Tromey  <[EMAIL PROTECTED]>
>       PR classpath/22689:
>       * java/io/PrintWriter.java (closed): New field.
>       (checkError): Only flush if stream not closed.
>       (close): Set 'closed'.
>
> [ ... in close() ... ]
>      try
>        {
> -     out.close();
> +        out.close();
> +        closed = true;
>        }
>      catch (IOException ex)
>        {

Shouldn't this be conditional on closed? And shouldn't closed be set to
true even if out.close() throws an IOException?

Cheers,

Mark

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to