Hi,

These are 2 more changes that were done on the old Classpath used in SableVM
but are not currently found in the most recent Classpath.

I think Etienne did these two changes, I'm not sure.  Maybe Etienne
could answer your questions on these.

java_io_File.c: Basically, it frees a local ref, I guess otherwise it may
run out of local refs for a long directory listing.

ObjectInputStream.java: I don't understand ClassLoader enough to explain this
one.



David



---

David Bélanger
Graduate Student
School of Computer Science
McGill University
Office: MC226

Web page:   http://www.cs.mcgill.ca/~dbelan2/
Public key: http://www.cs.mcgill.ca/~dbelan2/public_key.txt

Index: native/jni/java-io/java_io_File.c
===================================================================
--- native/jni/java-io/java_io_File.c   (revision 1011)
+++ native/jni/java-io/java_io_File.c   (revision 1076)
@@ -682,6 +682,7 @@
 
       /* save into array */
       (*env)->SetObjectArrayElement(env, filearray, i, str);
+      (*env)->DeleteLocalRef (env, str);
     }
 
   /* free resources */
Index: java/io/ObjectInputStream.java
===================================================================
--- java/io/ObjectInputStream.java      (revision 1011)
+++ java/io/ObjectInputStream.java      (revision 1076)
@@ -559,7 +559,7 @@
     if (cl == null)
       return Class.forName (osc.getName ());
     else
-      return cl.loadClass (osc.getName ());
+      return Class.forName (osc.getName (), false, cl);
   }
 
   /**
_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath

Reply via email to