From 6b0f7de4e34c7da1ec7ae82e8e8a298823e50d72 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Tue, 13 May 2025 18:41:09 +0200 Subject: [PATCH] MINOR: mux-spop: Don't set SPOP connection state to FRAME_H after ACK parsing After the ACK frame was parsed, it is useless to set the SPOP connection state to SPOP_CS_FRAME_H state because this will be automatically handled by the demux function. If it is not an issue, but this will simplify changes for the next commit. --- src/mux_spop.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mux_spop.c b/src/mux_spop.c index 87256ee50..d5149e6c1 100644 --- a/src/mux_spop.c +++ b/src/mux_spop.c @@ -1984,7 +1984,6 @@ static int spop_conn_handle_ack(struct spop_conn *spop_conn, struct spop_strm *s spop_strm->flags |= SPOP_SF_ACK_RCVD; TRACE_PROTO("SPOP AGENT ACK frame rcvd", SPOP_EV_RX_FRAME|SPOP_EV_RX_ACK, spop_conn->conn, spop_strm, 0, (size_t[]){sent}); - spop_conn->state = SPOP_CS_FRAME_H; TRACE_LEAVE(SPOP_EV_RX_FRAME|SPOP_EV_RX_ACK, spop_conn->conn, spop_strm); return 1;