Merge pull request #17 from paulpatience/master

Use fewer headers and change i to uint16_t
OpenCyphal · May 12, 2016 · ba523d0 · ba523d0
2 parents 34195e7 + 27b3c87
commit ba523d0
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 9 deletions.
diff --git a/src/canard.c b/src/canard.c
@@ -10,6 +10,7 @@
 #include "canard.h"
 #include "canard_internals.h"
 #include <inttypes.h>
+#include <string.h>
 
 
 #define CANARD_MAKE_TRANSFER_DESCRIPTOR(data_type_id, transfer_type, \
@@ -299,7 +300,7 @@ void canardHandleRxFrame(CanardInstance* ins, const CanardCANFrame* frame, uint6
         uint16_t middle_len = 0;
         if (rxstate->payload_len < CANARD_RX_PAYLOAD_HEAD_SIZE)
         {
-            int i;
+            uint16_t i;
             for (i=rxstate->payload_len, tail_offset=0; i<CANARD_RX_PAYLOAD_HEAD_SIZE && tail_offset<frame->data_len-1; i++, tail_offset++)
             {
                 rxstate->buffer_head[i] = frame->data[tail_offset];

diff --git a/src/canard.h b/src/canard.h
@@ -15,13 +15,8 @@
 extern "C" {
 #endif
 
-#include <stdio.h>
-#include <unistd.h>
 #include <stdint.h>
-#include <stdlib.h>
 #include <stddef.h>
-#include <string.h>
-#include <math.h>
 #include <stdbool.h>
 
 /** The size of a memory block in bytes. */

diff --git a/src/canard_internals.h b/src/canard_internals.h
@@ -25,7 +25,7 @@
 #ifndef CANARD_INTERNALS_H
 #define CANARD_INTERNALS_H
 
-#include <unistd.h>
+#include <stdint.h>
 
 #ifdef __cplusplus
 extern "C" {

diff --git a/src/canard_test.c b/src/canard_test.c
@@ -12,9 +12,7 @@
 #include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
-#include <math.h>
 #include <stdbool.h>
-#include <pthread.h>
 #include "canard.h"
 // #include "canard_debug.h"
 /*