MINOR: htx: rename htx_append_blk_value() to htx_add_data_atonce()

This function is now dedicated to data blocks, and we'll soon need to
access it from outside in a rare few cases. Let's rename it and export
it.

diff --git a/include/common/htx.h b/include/common/htx.h
index a6065a6e3704fa42f7a81575375bb7fe4315a505..463f7299bf511951591252a470563af8222277b8 100644 (file)
--- a/include/common/htx.h
+++ b/include/common/htx.h
@@ -185,6 +185,7 @@ struct htx_blk *htx_add_header(struct htx *htx, const struct ist name, const str
 struct htx_blk *htx_add_blk_type_size(struct htx *htx, enum htx_blk_type type, uint32_t blksz);
 struct htx_blk *htx_add_all_headers(struct htx *htx, const struct http_hdr *hdrs);
 struct htx_blk *htx_add_endof(struct htx *htx, enum htx_blk_type type);
+struct htx_blk *htx_add_data_atonce(struct htx *htx, const struct ist data);
 struct htx_blk *htx_add_data(struct htx *htx, const struct ist data);
 struct htx_blk *htx_add_trailer(struct htx *htx, const struct ist tlr);
 struct htx_blk *htx_add_data_before(struct htx *htx, const struct htx_blk *ref, const struct ist data);

diff --git a/src/htx.c b/src/htx.c
index 7bdeec1c37e084cdb46d5f7ba8d4c9b2ae1ee2f4..2aeb077f2f5ee1fa9130fa534d7fa5866044ae86 100644 (file)
--- a/src/htx.c
+++ b/src/htx.c
@@ -347,13 +347,13 @@ struct htx_ret htx_drain(struct htx *htx, uint32_t count)
 }
 
 /* Tries to append data to the last inserted block, if the type matches and if
- * there is enough non-wrapping space. Only DATA content can be appended. If the
- * append fails, a new block is inserted. If an error occurred, NULL is
+ * there is enough space to take it all. If the space wraps, the buffer is
+ * defragmented and a new block is inserted. If an error occurred, NULL is
  * returned. Otherwise, on success, the updated block (or the new one) is
- * returned.
-*/
-static struct htx_blk *htx_append_blk_value(struct htx *htx, enum htx_blk_type type,
-                                           const struct ist data)
+ * returned. Due to its nature this function can be expensive and should be
+ * avoided whenever possible.
+ */
+struct htx_blk *htx_add_data_atonce(struct htx *htx, const struct ist data)
 {
        struct htx_blk *blk, *tailblk, *headblk, *frtblk;
        struct ist v;
@@ -366,10 +366,6 @@ static struct htx_blk *htx_append_blk_value(struct htx *htx, enum htx_blk_type t
        if (data.len > htx_free_data_space(htx))
                return NULL;
 
-       /* Append only DATA */
-       if (type != HTX_BLK_DATA)
-               goto add_new_block;
-
        /* get the tail and head block */
        tailblk = htx_get_tail_blk(htx);
        headblk = htx_get_head_blk(htx);
@@ -378,7 +374,7 @@ static struct htx_blk *htx_append_blk_value(struct htx *htx, enum htx_blk_type t
 
        /* Don't try to append data if the last inserted block is not of the
         * same type */
-       if (type != htx_get_blk_type(tailblk))
+       if (htx_get_blk_type(tailblk) != HTX_BLK_DATA)
                goto add_new_block;
 
        /*
@@ -412,7 +408,7 @@ static struct htx_blk *htx_append_blk_value(struct htx *htx, enum htx_blk_type t
 
   add_new_block:
        /* FIXME: check tlr.len (< 256MB) */
-       blk = htx_add_blk(htx, type, data.len);
+       blk = htx_add_blk(htx, HTX_BLK_DATA, data.len);
        if (!blk)
                return NULL;
 
@@ -766,7 +762,7 @@ struct htx_blk *htx_add_endof(struct htx *htx, enum htx_blk_type type)
  */
 struct htx_blk *htx_add_data(struct htx *htx, const struct ist data)
 {
-       return htx_append_blk_value(htx, HTX_BLK_DATA, data);
+       return htx_add_data_atonce(htx, data);
 }
 
 /* Adds an HTX block of type TLR in <htx>. It returns the new block on