Refactor 'mcuicom_send' to elliminate the second argument.

The first argument is a pointer to the data to be sent and the second
argument represents the size of that data. After refactoring, the
function no longer needs the size of the data to be provided, since it now
sends bytes until a null terminator is found.

gpcore/shell.c

@@ -6,7 +6,6 @@
 #include "self_test.h"
 #include "../mcuicom.h"
 #include "mctlcom.h"
-#include "../macros.h"
 
 #include "../debug.h"
 //#ifndef NDEBUG
@@ -1754,7 +1753,8 @@ inline void hostcmd_pa(void)
 				buf[0] = 'R';
 				buf[1] = param1.value.letter;
 				buf[2] = *CMDEND;
-				mcuicom_send(buf, 3);
+				buf[3] = '\0';
+				mcuicom_send(buf);
 				// Get response (motor steps) and re-send it to the host PC
 				size = mctlcom_get_response(buf, size);
 				hostcom_send(buf, size);
@@ -2965,7 +2965,7 @@ inline void hostcmd_mc(void)
 	}
 	else
 	{
-		mcuicom_send("MC" CMDEND, STRLEN("MC" CMDEND));
+		mcuicom_send("MC" CMDEND);
 	}
 }
 
@@ -3007,7 +3007,7 @@ inline void hostcmd_mi(void)
 	}
 	else
 	{
-		mcuicom_send("MI" CMDEND, STRLEN("MI" CMDEND));
+		mcuicom_send("MI" CMDEND);
 	}
 }
 
@@ -3040,7 +3040,7 @@ inline void hostcmd_mm(void)
 					buf[0] = 'S';
 					buf[1] = param1.value.letter;
 					buf[2] = *CMDEND;
-					mcuicom_send(buf, 3);
+					mcuicom_send(buf);
 				}
 				else
 				{
@@ -3258,7 +3258,7 @@ inline void hostcmd_pd(void)
 
 									// Send MCUICOM command AA, AB, ..., AF depending on motor letter (param 1)
 									snprintf(buf, size, "A%c,%d%c", param1.value.letter, intparam2, *CMDEND);
-									mcuicom_send(buf, strlen(buf));
+									mcuicom_send(buf);
 								}
 								else
 								{
@@ -3345,7 +3345,7 @@ inline void hostcmd_pr(void)
 
 									// Send MCUICOM command BA, BB, ..., BF depending on motor letter (param 1)
 									snprintf(buf, size, "B%c,%d%c", param1.value.letter, intparam2, *CMDEND);
-									mcuicom_send(buf, strlen(buf));
+									mcuicom_send(buf);
 								}
 								else
 								{
@@ -3444,7 +3444,7 @@ inline void hostcmd_px(void)
 									const int size = 64;
 									char buf[size];
 									snprintf(buf, size, "C%c,%.2f%c", param1.value.letter, (double) floatparam2, *CMDEND);
-									mcuicom_send(buf, strlen(buf));
+									mcuicom_send(buf);
 								}
 								else
 								{
@@ -3545,7 +3545,7 @@ inline void hostcmd_py(void)
 									const int size = 64;
 									char buf[size];
 									snprintf(buf, size, "D%c,%.2f%c", param1.value.letter, (double) floatparam2, *CMDEND);
-									mcuicom_send(buf, strlen(buf));
+									mcuicom_send(buf);
 								}
 								else
 								{

mcuicom.c

@@ -44,10 +44,10 @@ inline void mcuicom_setup(void)
 	U1STAbits.UTXEN = 1;
 }
 
-int mcuicom_send(const char * const data, const int size)
+int mcuicom_send(const char * const data)
 {
 	int sent;
-	for (sent = 0; sent < size; ++sent)
+	for (sent = 0; data[sent] != '\0'; ++sent)
 	{
 		while (U1STAbits.UTXBF); // Wait while the transmit buffer is full
 		U1TXREG = data[sent];

mcuicom.h

@@ -36,7 +36,7 @@ extern buffer_t mcuicom_rcv_buf;
  */
 inline void mcuicom_setup(void);
 
-int mcuicom_send(const char * const data, const int size);
+int mcuicom_send(const char * const data);
 
 /**
  * Copy the first fully received GPMCU command from the receive buffer to the

motorctl/gpcorecom.c

@@ -613,48 +613,47 @@ void interpret_cmd(void)
  ******************************************************************************/
 
 #include <stdio.h>
-#include <string.h>
 
 inline void read_encoder_a(void)
 {
 	char buf[64];
 	snprintf(buf, 64, "%d%c", motor_steps[MOTOR_A], *CMDEND);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void read_encoder_b(void)
 {
 	char buf[64];
 	snprintf(buf, 64, "%d%c", motor_steps[MOTOR_B], *CMDEND);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void read_encoder_c(void)
 {
 	char buf[64];
 	snprintf(buf, 64, "%d%c", motor_steps[MOTOR_C], *CMDEND);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void read_encoder_d(void)
 {
 	char buf[64];
 	snprintf(buf, 64, "%d%c", motor_steps[MOTOR_D], *CMDEND);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void read_encoder_e(void)
 {
 	char buf[64];
 	snprintf(buf, 64, "%d%c", motor_steps[MOTOR_E], *CMDEND);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void read_encoder_f(void)
 {
 	char buf[64];
 	snprintf(buf, 64, "%d%c", motor_steps[MOTOR_F], *CMDEND);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void stop_motor_a(void)
@@ -1050,47 +1049,47 @@ inline void set_cartesian_abs_x(void)
 	// TODO: implement
 	char buf[64];
 	snprintf(buf, 64, "set_cartesian_abs_x: %f\n", (double) cartesian_desired_pos[COORD_X]);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void set_cartesian_abs_y(void)
 {
 	// TODO: implement
 	char buf[64];
 	snprintf(buf, 64, "set_cartesian_abs_y: %f\n", (double) cartesian_desired_pos[COORD_Y]);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void set_cartesian_abs_z(void)
 {
 	// TODO: implement
 	char buf[64];
 	snprintf(buf, 64, "set_cartesian_abs_z: %f\n", (double) cartesian_desired_pos[COORD_Z]);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void set_cartesian_rel_x(void)
 {
 	// TODO: implement
 	char buf[64];
 	snprintf(buf, 64, "set_cartesian_rel_x: %f\n", (double) cartesian_desired_pos[COORD_X]);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void set_cartesian_rel_y(void)
 {
 	// TODO: implement
 	char buf[64];
 	snprintf(buf, 64, "set_cartesian_rel_y: %f\n", (double) cartesian_desired_pos[COORD_Y]);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void set_cartesian_rel_z(void)
 {
 	// TODO: implement
 	char buf[64];
 	snprintf(buf, 64, "set_cartesian_rel_z: %f\n", (double) cartesian_desired_pos[COORD_Z]);
-	mcuicom_send(buf, strlen(buf));
+	mcuicom_send(buf);
 }
 
 inline void move_independent(void)