Re: [Sablevm-developer] More on bootstrapping

[John L. <je...@pi...>]

> I'm free to modify java/lang/Runtime all I want as it is a VM specific
> file :-) It seems badly written too as it half uses ':' as a path
> seperator and half uses File.pathSeperatorChar. The System.setProperty
> -> Hashtable.hash -> Math.<clinit> -> System.loadLibrary ->
> Runtime.loadLibrary -> ... bootstrapping problem seems to be resolved by
> modifying Runtime.Runtime too :-) Anyway my modifications are attached
> as a patch. I'll commit them if anyone wants.
> 
> Ta,
> 
> Ian
> Index: sablepath-classes/src/java/lang/Runtime.java
> ===================================================================
> RCS file: /cvsroot/sablevm/sablepath-classes/src/java/lang/Runtime.java,v
> retrieving revision 1.1.1.2
> diff -r1.1.1.2 Runtime.java
> 45a46,47
> > 		char pathSeparatorChar = ':';
> > 
> 55c57
> < 			if(path.charAt(i) == ':')
> ---
> > 			if(path.charAt(i) == pathSeparatorChar)
> 63c65
> < 			int next = path.indexOf(File.pathSeparatorChar,current);
> ---
> > 			int next = path.indexOf(pathSeparatorChar,current);
> 164a167

If I understand it correctly, you're now hardcoding the separator in your Runtime file?

John Leuner