From ad1cc3df9c69444291ff52f6179d11a487f29576 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Mon, 27 Aug 2012 18:54:20 +0200 Subject: [PATCH] MINOR: channel: rename bi_full to channel_full as it checks the whole channel Since the function takes care of the forward count and involves more than buffer knowledge, rename it. --- include/proto/channel.h | 6 ++---- src/channel.c | 10 +++++----- src/stream_interface.c | 4 ++-- 3 files changed, 9 insertions(+), 11 deletions(-) diff --git a/include/proto/channel.h b/include/proto/channel.h index 4aa6a5722..6d4d3e739 100644 --- a/include/proto/channel.h +++ b/include/proto/channel.h @@ -107,7 +107,7 @@ static inline int buffer_max_len(const struct channel *buf) * close to happen. The test is optimized to avoid as many operations as * possible for the fast case and to be used as an "if" condition. */ -static inline int bi_full(const struct channel *b) +static inline int channel_full(const struct channel *b) { int rem = b->buf.size; @@ -228,8 +228,6 @@ static inline void bi_erase(struct channel *buf) buf->buf.i = 0; buf->flags &= ~BF_FULL; - if (bi_full(buf)) - buf->flags |= BF_FULL; } /* marks the buffer as "shutdown" ASAP for reads */ @@ -322,7 +320,7 @@ static inline void bo_skip(struct channel *buf, int len) if (buffer_len(&buf->buf) == 0) buf->buf.p = buf->buf.data; - if (!bi_full(buf)) + if (!channel_full(buf)) buf->flags &= ~BF_FULL; /* notify that some data was written to the SI from the buffer */ diff --git a/src/channel.c b/src/channel.c index 86fa81130..0e1af1349 100644 --- a/src/channel.c +++ b/src/channel.c @@ -128,7 +128,7 @@ int bo_inject(struct channel *buf, const char *msg, int len) buf->total += len; buf->flags &= ~BF_FULL; - if (bi_full(buf)) + if (channel_full(buf)) buf->flags |= BF_FULL; return -1; @@ -152,7 +152,7 @@ int bi_putchr(struct channel *buf, char c) *bi_end(&buf->buf) = c; buf->buf.i++; - if (bi_full(buf)) + if (channel_full(buf)) buf->flags |= BF_FULL; buf->flags |= BF_READ_PARTIAL; @@ -215,7 +215,7 @@ int bi_putblk(struct channel *buf, const char *blk, int len) } buf->flags &= ~BF_FULL; - if (bi_full(buf)) + if (channel_full(buf)) buf->flags |= BF_FULL; /* notify that some data was read from the SI into the buffer */ @@ -346,7 +346,7 @@ int buffer_replace2(struct channel *b, char *pos, char *end, const char *str, in b->flags &= ~BF_FULL; if (buffer_len(&b->buf) == 0) b->buf.p = b->buf.data; - if (bi_full(b)) + if (channel_full(b)) b->flags |= BF_FULL; return delta; @@ -384,7 +384,7 @@ int buffer_insert_line2(struct channel *b, char *pos, const char *str, int len) b->buf.i += delta; b->flags &= ~BF_FULL; - if (bi_full(b)) + if (channel_full(b)) b->flags |= BF_FULL; return delta; diff --git a/src/stream_interface.c b/src/stream_interface.c index 14e2cd5ce..2f3400dcc 100644 --- a/src/stream_interface.c +++ b/src/stream_interface.c @@ -727,7 +727,7 @@ static int si_conn_send_loop(struct connection *conn) b->flags |= BF_WRITE_PARTIAL; - if (likely(!bi_full(b))) + if (likely(!channel_full(b))) b->flags &= ~BF_FULL; if (!b->buf.o) { @@ -1080,7 +1080,7 @@ void si_conn_recv_cb(struct connection *conn) b->flags |= BF_READ_PARTIAL; b->total += ret; - if (bi_full(b)) { + if (channel_full(b)) { /* The buffer is now full, there's no point in going through * the loop again. */