cleanup with message data layout
authorBrendan Hansen <brendan.f.hansen@gmail.com>
Tue, 2 Aug 2022 03:13:15 +0000 (22:13 -0500)
committerBrendan Hansen <brendan.f.hansen@gmail.com>
Tue, 2 Aug 2022 03:13:15 +0000 (22:13 -0500)
src/debug/debug_thread.c

index 8ef322032fc9728e63cf335442c7c3431794d6a5..4d938daff8a2ee5284dd621654931144e1b50ffa 100644 (file)
@@ -19,13 +19,7 @@ struct msg_parse_ctx_t {
 };
 
 static void send_empty_response(debug_state_t *debug, unsigned int message_number) {
-    char message_buf[5];
-    message_buf[0] = 1;
-    message_buf[1] = (message_number >> 0)  & 0xff;
-    message_buf[2] = (message_number >> 8)  & 0xff;
-    message_buf[3] = (message_number >> 16) & 0xff;
-    message_buf[4] = (message_number >> 24) & 0xff;
-    send(debug->client_fd, message_buf, 5, 0);
+    send(debug->client_fd, &message_number, 4, 0);
 }
 
 static void process_command(debug_state_t *debug, struct msg_parse_ctx_t *ctx) {
@@ -69,10 +63,9 @@ static void process_message(debug_state_t *debug, char *msg, unsigned int bytes_
     ctx.bytes_read = bytes_read;
 
     while (ctx.offset < ctx.bytes_read) {
-        u8 type = *(u8 *) &ctx.data[ctx.offset];
-        ctx.offset++;
+        u32 msg_id = *(u32 *) &ctx.data[ctx.offset];
 
-        if (type == 0) {
+        if (msg_id != 0xffffffff) {
             process_command(debug, &ctx);
         } else {
             process_response(debug, &ctx);