Drop the pretense of C89, put declarations where they make sense

[model-railway-devices.git] / arducontrol / interface.c

diff --git a/arducontrol/interface.c b/arducontrol/interface.c
index 2c55e2af634eca747a7d207db38235a037c24cd3..014e8e98bcaa0ac5eacbeb061f1a38754e1a2ae5 100644 (file)
--- a/arducontrol/interface.c
+++ b/arducontrol/interface.c
@@ -22,11 +22,10 @@ void interface_init(void)
 
 void interface_check(void)
 {
-       uint8_t count;
        if(serial_read_overrun())
                interface_send1(RECEIVE_OVERRUN);
 
-       count = serial_read_available();
+       uint8_t count = serial_read_available();
        if(count>0)
        {
                PORTB |= 0x01;
@@ -47,10 +46,9 @@ void interface_check(void)
 
                if(cmd_read_pos<cmd_length)
                {
-                       uint8_t i;
                        if(cmd_read_pos+count>cmd_length)
                                count = cmd_length-cmd_read_pos;
-                       for(i=0; i<count; ++i)
+                       for(uint8_t i=0; i<count; ++i)
                                cmd_buffer[cmd_read_pos++] = serial_read();
 
                        if(cmd_read_pos>=cmd_length)
@@ -85,10 +83,8 @@ static uint8_t dispatch_command(const uint8_t *cmd, uint8_t length)
 
 void interface_send(const uint8_t *cmd, uint8_t length)
 {
-       uint8_t i;
-
        serial_write(~length);
-       for(i=0; i<length; ++i)
+       for(uint8_t i=0; i<length; ++i)
                serial_write(cmd[i]);
 }