summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Wu <peter@lekensteyn.nl>2015-06-12 10:55:08 +0200
committerPeter Wu <peter@lekensteyn.nl>2015-06-12 10:55:08 +0200
commitca866b216b49f5f61c2698ee0026627e9af371f5 (patch)
tree42437340fa0687f5e6548ab6d785412a03ea4a69
parent8bf816608589e996bc8b38d2d562d8abdead68cc (diff)
downloadcode-ca866b216b49f5f61c2698ee0026627e9af371f5.tar.gz
serial: Fix signed overflow warning
-rw-r--r--Venus_Skeleton/comm.ino9
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;
}
}