auto-sync

firmware/console/binary/tunerstudio.cpp

@@ -585,7 +585,8 @@ static void handleGetText(ts_channel_s *tsChannel) {
 static void handleExecuteCommand(ts_channel_s *tsChannel, char *data, int incomingPacketSize) {
 	tunerStudioWriteCrcPacket(tsChannel, TS_RESPONSE_COMMAND_OK, NULL, 0);
 	data[incomingPacketSize] = 0;
-	(console_line_callback)(data);
+	char *trimmed = efiTrim(data);
+	(console_line_callback)(trimmed);
 }
 
 /**

firmware/console/console_io.cpp

@@ -187,7 +187,9 @@ static msg_t consoleThreadThreadEntryPoint(void *arg) {
 			continue;
 		}
 
-		(console_line_callback)(consoleInput);
+		char *trimmed = efiTrim(consoleInput);
+
+		(console_line_callback)(trimmed);
 
 		if (consoleInBinaryMode) {
 			// switch to binary protocol

unit_tests/main.cpp

@@ -27,11 +27,12 @@
 
 #include "test_signal_executor.h"
 
+#include "test_util.h"
+
 extern "C"
 {
 #include "map_resize.h"
 #include "test_event_registry.h"
-#include "test_util.h"
 }
 
 #include "engine_math.h"
@@ -117,6 +118,7 @@ int main(void) {
 	testSensors();
 	testCyclicBuffer();
 	testCrc();
+	testMisc();
 
 	testSignalExecutor();
 

unit_tests/test_util.cpp

@@ -351,7 +351,14 @@ void testFLStack(void) {
 
 }
 
+static char buff[32];
+
 void testMisc(void) {
+	print("******************************************* testMisc\r\n");
+	strcpy(buff, "  ab  ");
+	// we need a mutable array here
+	assertTrue(strEqual("ab", efiTrim(buff)));
+
 //	assertEquals(true, strEqual("spa3", getPinName(SPARKOUT_3_OUTPUT)));
 //	assertEquals(SPARKOUT_12_OUTPUT, getPinByName("spa12"));
 }

unit_tests/test_util.h

@@ -8,12 +8,8 @@
 #ifndef TEST_CYCLIC_BUFFER_H_
 #define TEST_CYCLIC_BUFFER_H_
 
-#ifdef __cplusplus
-extern "C"
-{
-#endif /* __cplusplus */
-
 void testCrc(void);
+void testMisc(void);
 void testCyclicBuffer(void);
 void testOverflow64Counter(void);
 void testHistogram(void);
@@ -24,9 +20,4 @@ void testFLStack(void);
 void testMisc(void);
 void testMenuTree(void);
 
-#ifdef __cplusplus
-}
-#endif /* __cplusplus */
-
-
 #endif /* TEST_CYCLIC_BUFFER_H_ */