This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

FYI: ObjectOutputStream fixlet


Hi,

The recent patch that put the code for writing a Class descriptor in its
own method missed a break statement that was in the original version.
This patch puts it back.

2003-07-09  Mark Wielaard  <mark@klomp.org>

        * java/io/ObjectOutputStream.java (writeObject): break after
        calling writeClassDescriptor().

This fixes a couple of Mauve regressions. I will put the fix also in
Classpath.

BTW Do people still have trouble running Mauve?

Cheers,

Mark
Index: java/io/ObjectOutputStream.java
===================================================================
RCS file: /cvs/gcc/gcc/libjava/java/io/ObjectOutputStream.java,v
retrieving revision 1.16
diff -u -r1.16 ObjectOutputStream.java
--- java/io/ObjectOutputStream.java	25 Jun 2003 06:31:59 -0000	1.16
+++ java/io/ObjectOutputStream.java	9 Jul 2003 10:49:31 -0000
@@ -224,7 +224,10 @@
 	      }
 
 	    if (obj instanceof ObjectStreamClass)
-	      writeClassDescriptor ((ObjectStreamClass) obj);
+	      {
+		writeClassDescriptor ((ObjectStreamClass) obj);
+		break;
+	      }
 
 	    if ((replacementEnabled || obj instanceof Serializable)
 		&& ! replaceDone)

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]