Patch: invalid code removed

Tom Tromey tromey@cygnus.com
Tue Sep 21 12:49:00 GMT 1999


I'm checking in the appended patch, which fixes some invalid code in
libgcj.

1999-09-21  Tom Tromey  <tromey@cygnus.com>

	* java/io/OutputStreamWriter.java (OutputStreamWriter): Don't
	refer to `this' before calling superclass constructor.
	* java/io/PrintStream.java (PrintStream): Don't refer to `this'
	before calling superclass constructor.

Tom

Index: java/io/OutputStreamWriter.java
===================================================================
RCS file: /cvs/java/libgcj/libjava/java/io/OutputStreamWriter.java,v
retrieving revision 1.2
diff -u -r1.2 OutputStreamWriter.java
--- OutputStreamWriter.java	1999/04/16 18:35:01	1.2
+++ OutputStreamWriter.java	1999/09/21 19:44:32
@@ -32,9 +32,11 @@
 
   private OutputStreamWriter(OutputStream out, UnicodeToBytes encoder)
   {
-    super((this.out = (out instanceof BufferedOutputStream
-		       ? (BufferedOutputStream) out
-		       : new BufferedOutputStream(out, 250))));
+    BufferedOutputStream buf = (out instanceof BufferedOutputStream
+				? (BufferedOutputStream) out
+				: new BufferedOutputStream(out, 250));
+    super (buf);
+    this.out = buf;
     this.converter = encoder;
   } 
 
Index: java/io/PrintStream.java
===================================================================
RCS file: /cvs/java/libgcj/libjava/java/io/PrintStream.java,v
retrieving revision 1.4
diff -u -r1.4 PrintStream.java
--- PrintStream.java	1999/04/18 22:22:44	1.4
+++ PrintStream.java	1999/09/21 19:44:32
@@ -238,9 +238,11 @@
 
   public PrintStream (OutputStream out, boolean af)
   {
-    super ((this.out = (out instanceof BufferedOutputStream
-			 ? (BufferedOutputStream) out
-			 : new BufferedOutputStream(out, 250))));
+    BufferedOutputStream buf = (out instanceof BufferedOutputStream
+				? (BufferedOutputStream) out
+				: new BufferedOutputStream(out, 250));
+    super (buf);
+    this.out = buf;
     converter = UnicodeToBytes.getDefaultEncoder();
     error = false;
     auto_flush = af;


More information about the Java-patches mailing list