On Sun, 2007-04-08 at 22:41 +0100, Robert Lougher wrote: > Strangely enough, I _did_ fix this in JamVM 1.4.4, but then removed > the checks again in JamVM 1.4.5! > > http://cvs.berlios.de/cgi-bin/viewcvs.cgi/jamvm/jamvm/src/reflect.c.diff?r1=1.9&r2=1.10 > > If I remember the reason for removing it again was that it broke some > stuff. I had a quick look at cacao and gcj, and found they didn't do > the check, and not having the time at that point to look into it > properly just backed the change out. I'll see if I can find what > broke.
We had a similar check in there http://b2.complang.tuwien.ac.at/cgi-bin/viewcvs.cgi/cacao/trunk/src/vm/access.c?rev=7418&view=diff&r1=7418&r2=7417&p1=cacao/trunk/src/vm/access.c&p2=/cacao/trunk/src/vm/access.c and I removed it because of a bug. But I too can't remember what bug that was :-/ - twisti