Hi Willy,

find my (probably :-) ) final CLEANUP series for 2.5.

Regarding the final patch:

'chunk_strncat()' appears to be completely redundant, it simply passes through
the arguments and even takes an int instead of a size_t. Should it be removed?

Best regards
Tim Düsterhus

Tim Duesterhus (6):
  DEV: coccinelle: Add rule to use `isttrim()` where possible
  CLEANUP: Apply ist.cocci
  DEV: coccinelle: Add rule to use `chunk_istcat()` instead of
    `chunk_memcat()`
  DEV: coccinelle: Add rule to use `chunk_istcat()` instead of
    `chunk_strncat()`
  CLEANUP: Apply ist.cocci
  CLEANUP: chunk: Remove duplicated chunk_Xcat implementation

 dev/coccinelle/ist.cocci | 24 +++++++++++++++++++++++
 include/haproxy/chunk.h  | 41 +++++++++++++---------------------------
 src/cache.c              |  5 ++---
 src/flt_trace.c          |  3 +--
 src/hlua.c               |  6 ++----
 src/http_ana.c           |  3 +--
 src/http_fetch.c         |  2 +-
 src/http_htx.c           |  4 ++--
 src/log.c                |  6 ++----
 src/mux_fcgi.c           | 10 +++++-----
 src/tcpcheck.c           |  4 ++--
 11 files changed, 55 insertions(+), 53 deletions(-)

-- 
2.33.1


Reply via email to