[cp-patches] FYI: Fix bug in static Byte constructor

Christian Thalinger twisti at complang.tuwien.ac.at
Mon Apr 21 14:48:26 UTC 2008


On Mon, 2008-04-21 at 14:34 +0000, Andrew John Hughes wrote:
> Indeed you did suggest it, but didn't give me a good reason to do it... ;)

True :-)

> 
> Anyway, here's the patch to change to an int.  I haven't changed the other 
> two, because it also means introducing a cast in the loop body.  If it's 
> still worthwhile, given this, I'll add that too.

I think it doesn't matter, performance wise, anyways.  It's a static
class initializer, running only once.  An int would just be more
obvious.

Btw, the fix works.  Thanks for commiting it.

- twisti




More information about the Classpath-patches mailing list