Mark Wielaard wrote:
> On Mon, 2005-07-25 at 18:17 +0200, Jeroen Frijters wrote:
> > Is this an important scenario? I thought you long haired 
> types didn't
> > like binary distributions ;-)
> 
> He he. Sure we don't. But if you follow the harmony effort of the
> cleanly shaved apache types you will have noticed that they might
> actually be paranoid enough to not look at or use something that comes
> with source code :)
> 
> But I do care about binary installations!
> I have here two runtimes (jamvm and kissme) who both use the same GNU
> Classpath installation, but both overlay the VM classes differently. I
> would actually like that to be true for some of the others I have
> installed here too.

That's a good point. I've removed the final modifier, that should do the
trick and still allow VMs that replace the class to make the field
final.

Regards,
Jeroen

2005-07-26  Jeroen Frijters  <[EMAIL PROTECTED]>

        * vm/reference/java/lang/VMClassLoader.java
        (USE_VM_CACHE): Removed final modifier.
Index: vm/reference/java/lang/VMClassLoader.java
===================================================================
RCS file: 
/cvsroot/classpath/classpath/vm/reference/java/lang/VMClassLoader.java,v
retrieving revision 1.28
diff -u -r1.28 VMClassLoader.java
--- vm/reference/java/lang/VMClassLoader.java   25 Jul 2005 14:28:42 -0000      
1.28
+++ vm/reference/java/lang/VMClassLoader.java   26 Jul 2005 06:45:09 -0000
@@ -285,8 +285,10 @@
 
   /**
    * Set this field to true if the VM wants to keep its own cache.
+   * Note that this field is not final, to allow VMs to have a
+   * different setting without having to recompile ClassLoader.java.
    */
-  static final boolean USE_VM_CACHE = false;
+  static boolean USE_VM_CACHE = false;
 
   /**
    * If the VM wants to keep its own cache, this method can be replaced.
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to