[PATCH 0/9] Some more static checker fixes

[David Gibson <david@gibson.dropbear.id.au>]

This series includes a number of fixes related to the static checkers:

 * Fedora 40 has updated to Cppcheck 2.14.1 which introduces some new
   warnings.  Fix them.

 * Jon's recent patch caused a small cppcheck regression.  I assume
   neither Jon nor Stefano is using sufficiently recent cppcheck
   versions to catch it.  Fix that too.

 * We were disabling the bugprone-macro parentheses check in
   clang-tidy.  I don't think that's a good idea.  Re-enable it and
   fix existing warnings.

 * It might also be a good idea to enable the
   bugprone-narrowing-conversions check.  Fix a number of issues
   across the tree which, amongst other things trigger that warning.
   There are lots of other places that trigger the warning which I
   haven't fixed yet, so don't enable it yet.

David Gibson (9):
  tcp: Make pointer const in tcp_revert_seq
  udp: Make rport calculation more local
  cppcheck: Suppress constParameterCallback errors
  Remove pointless macro parameters in CALL_PROTO_HANDLER
  clang-tidy: Enable the bugprone-macro-parentheses check
  util: Use unsigned indices for bits in bitmaps
  conf: Safer parsing of MAC addresses
  lineread: Use ssize_t for line lengths
  util: Use 'long' to represent millisecond durations

 Makefile     |  3 +--
 conf.c       | 55 +++++++++++++++++++++++++++++++++++-----------------
 lineread.c   | 10 ++++------
 lineread.h   |  7 ++++---
 passt.c      |  6 +++---
 tap.c        | 37 ++++++++++++++++++-----------------
 tcp.c        | 10 +++++-----
 tcp_splice.c |  4 ++--
 udp.c        |  3 +--
 util.c       | 10 +++++-----
 util.h       |  8 ++++----
 11 files changed, 85 insertions(+), 68 deletions(-)

-- 
2.45.2