]> git.ipfire.org Git - thirdparty/haproxy.git/commit
CLEANUP: mux-h2: remove the unused "full" variable in h2_frt_transfer_data()
authorWilly Tarreau <w@1wt.eu>
Thu, 24 Oct 2024 12:11:06 +0000 (14:11 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 24 Oct 2024 14:12:46 +0000 (16:12 +0200)
commitdb76949cff312619a1aa76c10622362eb5b9997b
treec95561e6de56b45575deac34f720a3971648e86c
parentf163cbfb7f893a06d158880a753cad01908143d8
CLEANUP: mux-h2: remove the unused "full" variable in h2_frt_transfer_data()

During 11th and 12th iteration of the development cycle for the H2 auto
rx window, several approaches were attempted to figure if another buffer
could be allocated or not. One of them consisted in looping back to the
beginning of the function requesting a new buffer slot and getting one
if the buffer was either apparently or confirmed full. The latest one
consisted in directly allocating the next buffer from the two places
where it's found to be proven full, instead of checking with the now
defunct h2s_may_get_rxbuf() if we were allowed to get once an loop.
That approach was retained. In this case the "full" variabled is no
longer needed, so let's get rid of it because the construct looks bogus
and confuses coverity (and possibly code readers as the intent is unclear
compared to the code).
src/mux_h2.c