commit e0d0b4089dd1c1dad94580de9de80985730da593
author Willy Tarreau <w@1wt.eu> Thu Aug 08 08:06:27 2019 +0200
committer Willy Tarreau <w@1wt.eu> Thu Aug 08 08:07:45 2019 +0200
tree f036d084bf36bbfa2af00e264bae5a6ba7e1c6da
parent 3b091f80aa66d538ea65a1fd7e60acfb37c5421a

CLEANUP: buffer: replace b_drop() with b_free()

Since last commit there's no point anymore in having two variants of the
same function, let's switch to b_free() only. __b_drop() was renamed to
__b_free() for obvious consistency reasons.

doc/internals/buffer-api.txt

diff --git a/doc/internals/buffer-api.txt b/doc/internals/buffer-api.txt
index abb5e9f..6d192c1 100644
--- a/doc/internals/buffer-api.txt
+++ b/doc/internals/buffer-api.txt
@@ -567,10 +567,7 @@
                     |                  | never calls malloc(), so it can fail
                     |                  | even if some memory is available
 --------------------+------------------+---------------------------------------
-__b_drop            | buffer *buf      | releases <buf> which must be allocated
-                    | ret: void        | and marks it empty
---------------------+------------------+---------------------------------------
-b_drop              | buffer *buf      | releases <buf> only if it is allocated
+__b_free            | buffer *buf      | releases <buf> which must be allocated
                     | ret: void        | and marks it empty
 --------------------+------------------+---------------------------------------
 b_free              | buffer *buf      | releases <buf> only if it is allocated

include/common/buffer.h

diff --git a/include/common/buffer.h b/include/common/buffer.h
index 34a3e95..0d44ec8 100644
--- a/include/common/buffer.h
+++ b/include/common/buffer.h
@@ -112,7 +112,7 @@
 /* Releases buffer <buf> (no check of emptiness). The buffer's head is marked
  * empty.
  */
-static inline void __b_drop(struct buffer *buf)
+static inline void __b_free(struct buffer *buf)
 {
 	char *area = buf->area;
 
@@ -125,16 +125,10 @@
 }
 
 /* Releases buffer <buf> if allocated, and marks it empty. */
-static inline void b_drop(struct buffer *buf)
-{
-	if (buf->size)
-		__b_drop(buf);
-}
-
-/* Releases buffer <buf> if allocated, and marks it empty. */
 static inline void b_free(struct buffer *buf)
 {
-	b_drop(buf);
+	if (buf->size)
+		__b_free(buf);
 }
 
 /* Ensures that <buf> is allocated. If an allocation is needed, it ensures that

src/stream.c

diff --git a/src/stream.c b/src/stream.c
index a135dcb..913de63 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -404,8 +404,8 @@
 		HA_SPIN_UNLOCK(BUF_WQ_LOCK, &buffer_wq_lock);
 	}
 	if (s->req.buf.size || s->res.buf.size) {
-		b_drop(&s->req.buf);
-		b_drop(&s->res.buf);
+		b_free(&s->req.buf);
+		b_free(&s->res.buf);
 		offer_buffers(NULL, tasks_run_queue);
 	}