- noticed two loops looping over same range with independent code, combined them
- removed unneccessary casts from int to int (I left in casts from char to int as the code may rely on these casts in some obscure way, I guessed that a cast from int to int was truly unneccesary though)
- use FileUtils to close stream
question: Do we assume that when arrays are initialised they have an undefined value? eg: boolean[] test = new array[10];
test[0] == null?
Reading VM spec suggests that a boolean is initialised to false and that was my understanding too, but does this behaviour change when an array of primitives is created? I'm pretty sure it doesn't, but it may be a VM vendor issue so I left in the code that looped across arrays and set them all to false in case a particular VM doesn't do this.
http://java.sun.com/docs/books/vmspec/2nd-edition/html/Concepts.doc.html#15858
Kev
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]