From 362bc06f72bc3117b9cf186e0b2f6a21ffc0b96a Mon Sep 17 00:00:00 2001 From: Mark Liversedge Date: Sun, 4 Dec 2011 14:31:04 +0000 Subject: [PATCH] Use uint8_t* not void* in memcpy() Computrainer spinscan memcpy() used offsets but failed to declare the datatype used by pointers. Interestingly, some compilers will assume a void* points to a byte, whilst others will rightly complain. --- src/Computrainer.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Computrainer.cpp b/src/Computrainer.cpp index d0c515579..a9d17b514 100644 --- a/src/Computrainer.cpp +++ b/src/Computrainer.cpp @@ -373,8 +373,8 @@ void Computrainer::unpackTelemetry(int &ss1, int &ss2, int &ss3, int &buttons, i value12 = value8 | (b1&7)<<9 | (b3&2)<<7; if (buttons&64) { - memcpy((void*)spinScan, (void*)ss+3, 21); - memcpy((void*)spinScan+21, (void*)ss, 3); + memcpy((uint8_t*)spinScan, (uint8_t*)ss+3, 21); + memcpy((uint8_t*)spinScan+21, (uint8_t*)ss, 3); //for (pos=0; pos<24; pos++) fprintf(stderr, "%d, ", ss[pos]); //fprintf(stderr, "\n"); pos=0;