John Keiser wrote:
> Note: you are probably already doing this, but it's a good idea to look at
> the VM Integration Guide
> (http://www.gnu.org/software/classpath/doc/vmintegration.html) when messing
> around with core classes like this so you make sure not to break VMs (well,
> to make sure not to break Japhar anyway :). While ThreadGroup is not in the
> VM interface, there is a note about it in the guide. Just for future
> reference; the stuff you did was good, from first glances.
Yeah, I actually held off committing it to classpath when I realised this might
be a problem. It turns out I had used the "add(Thread)" method name from
libgcj, where as classpath's reference Thread.java expects an
"addThread(Thread)". The updated copy is attached.
regards
[ bryce ]
