In HardwareSerial, don't use int for buffer indices

The index attributes have been uint8_t for a while, so there is no point
in using int for local variables. This should allow the compiler to
generate slightly more efficient code, but (at least on gcc 4.8.2) it
also confuses the register allocator, causing this change to increase
code size by 2 bytes instead due to extra push/pop instructions (but
this will probably change in the future if the compiler improves).
Merge pull request #1863 from matthijskooijman/ide-1.5.x-serial-int

In HardwareSerial, don't use int for buffer indices