Re: [Sablevm-developer] More on bootstrapping

[Etienne M. G. <eti...@uq...>]

Hi Ian,

Just go ahead and check it in.  We certainly don't want system dependant stuff in there.

Etienne

On Thu, Sep 20, 2001 at 06:57:54PM +0000, Ian Rogers wrote:
> Hi,
> 
> 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
> > 	    


-- 
Etienne M. Gagnon                    http://www.info.uqam.ca/~egagnon/
SableVM:                                       http://www.sablevm.org/
SableCC:                                       http://www.sablecc.org/