From ea308bac290aefb438cb495105dcaffcc47b08dc Mon Sep 17 00:00:00 2001 From: ti_mo Date: Fri, 5 Jun 2026 00:41:30 +0200 Subject: [PATCH] flipped '\n' and char storing statement, to not write '\n' into buffer and then overwriting it with '\0' --- main.cpp | 15 ++++++++------- response_tester.cs | 7 ++++--- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/main.cpp b/main.cpp index d259991..eb60797 100644 --- a/main.cpp +++ b/main.cpp @@ -73,6 +73,14 @@ static void read_serial_line_task(void *params){ continue; } + if(c == '\n'){ + buf[it] = '\0'; + it = 0; + + xQueueSendToBack(serial_queue, buf, portMAX_DELAY); + continue; + } + if(it < MAX_COMMAND_LENGTH - 1){ buf[it++] = (char)c; } @@ -80,13 +88,6 @@ static void read_serial_line_task(void *params){ printf_sync("Command too long, discarding\n"); it = 0; } - - if(c == '\n'){ - buf[it] = '\0'; - it = 0; - - xQueueSendToBack(serial_queue, buf, portMAX_DELAY); - } } } diff --git a/response_tester.cs b/response_tester.cs index ec1a150..9ddbca5 100644 --- a/response_tester.cs +++ b/response_tester.cs @@ -42,7 +42,7 @@ namespace ResponseTester { Console.WriteLine("Starting response tester..."); - var portName = "COM30"; + var portName = "/dev/cu.usbmodem101"; var baudrate = 115200; using var serialPort = new SerialPort(portName, baudrate, Parity.None, 8, StopBits.One); @@ -72,7 +72,8 @@ namespace ResponseTester Console.WriteLine($"Sent command: {command.Trim()}"); counter++; - if (counter % burst == 0) { + if (counter % burst == 0) + { Thread.Sleep(10); } } @@ -98,4 +99,4 @@ namespace ResponseTester Console.WriteLine($"Average Latency: {averageLatency.TotalMilliseconds} ms"); } } -} \ No newline at end of file +}