diff options
author | Peter Wu <peter@lekensteyn.nl> | 2015-06-12 10:55:08 +0200 |
---|---|---|
committer | Peter Wu <peter@lekensteyn.nl> | 2015-06-12 10:55:08 +0200 |
commit | ca866b216b49f5f61c2698ee0026627e9af371f5 (patch) | |
tree | 42437340fa0687f5e6548ab6d785412a03ea4a69 | |
parent | 8bf816608589e996bc8b38d2d562d8abdead68cc (diff) | |
download | code-ca866b216b49f5f61c2698ee0026627e9af371f5.tar.gz |
serial: Fix signed overflow warning
-rw-r--r-- | Venus_Skeleton/comm.ino | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/Venus_Skeleton/comm.ino b/Venus_Skeleton/comm.ino index d60138e..7f9e422 100644 --- a/Venus_Skeleton/comm.ino +++ b/Venus_Skeleton/comm.ino @@ -6,6 +6,7 @@ #include "definitions.h" #include "dataTypes.h" #include "comm.h" +#include <stdint.h> serial_state_t serial_state = SERIAL_UNKNOWN; @@ -74,9 +75,9 @@ static void serial_write_len(unsigned len) { } // write remainder of data -static void serial_write_data(const char *data, unsigned len) { +static void serial_write_data(const uint8_t *data, unsigned len) { for (unsigned i = 0; i < len; i++) { - char c = data[i]; + uint8_t c = data[i]; if (c & 0x80) { Serial.write(DATA_ESCAPE); Serial.write(c & ~0x80); @@ -88,7 +89,7 @@ static void serial_write_data(const char *data, unsigned len) { void serial_print_debug(const char *str) { unsigned len = strlen(str); - const char c = DATA_DEBUG_PREFIX; + const uint8_t c = DATA_DEBUG_PREFIX; // only write data when the serial connection is ready if (serial_state != SERIAL_READY) { @@ -101,7 +102,7 @@ void serial_print_debug(const char *str) { // add one for the debug prefix serial_write_len(1 + datalen); serial_write_data(&c, 1); /* write prefix */ - serial_write_data(str, datalen); /* write actual data payload */ + serial_write_data((uint8_t *)str, datalen); /* write actual data payload */ len -= datalen; } } |