public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: Stefano Brivio <sbrivio@redhat.com>, passt-dev@passt.top
Cc: David Gibson <david@gibson.dropbear.id.au>
Subject: [PATCH 00/12] Minor fixups for or inspired by clangd and related tools
Date: Wed,  6 Nov 2024 10:25:16 +1100	[thread overview]
Message-ID: <20241105232528.1408144-1-david@gibson.dropbear.id.au> (raw)

I've been experimenting with Zed and clangd recently.  Currently it
generates an enormous number of largely spurious errors and warnings
on the passt code base.  Mostly that's due to its default
configurations not suiting us.  This series adds some configuration
that addresses a number of those warnings, though there remain many
more for now.

Some of the warnings also look reasonable, so I have a grab bag of
fixes or workarounds for some of those two.

David Gibson (12):
  clang: Add .clang-format file
  Makefile: Simplify exclusion of qrap from static checks
  clang: Move clang-tidy configuration from Makefile to .clang-tidy
  arch: Avoid explicit access to 'environ'
  flow: Correct type of flowside_at_sidx()
  netlink: RTA_PAYLOAD() returns int, not size_t
  Makefile: Move NETNS_RUN_DIR definition to C code
  seccomp: Simplify handling of AUDIT_ARCH
  Makefile: Use -DARCH for qrap only
  Makefile: Don't attempt to auto-detect stack size
  clang: Add rudimentary clangd configuration
  util: Remove unused ffsl() function

 .clang-format | 126 ++++++++++++++++++++++++++++++++++++++++++++++
 .clang-tidy   |  93 ++++++++++++++++++++++++++++++++++
 .clangd       |   3 ++
 Makefile      | 136 +++-----------------------------------------------
 arch.c        |   2 +-
 conf.c        |   2 +
 flow_table.h  |   2 +-
 netlink.c     |   4 +-
 seccomp.sh    |  14 +++++-
 util.h        |   5 +-
 10 files changed, 247 insertions(+), 140 deletions(-)
 create mode 100644 .clang-format
 create mode 100644 .clang-tidy
 create mode 100644 .clangd

-- 
2.47.0


             reply	other threads:[~2024-11-05 23:27 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-05 23:25 David Gibson [this message]
2024-11-05 23:25 ` [PATCH 01/12] clang: Add .clang-format file David Gibson
2024-11-05 23:25 ` [PATCH 02/12] Makefile: Simplify exclusion of qrap from static checks David Gibson
2024-11-05 23:25 ` [PATCH 03/12] clang: Move clang-tidy configuration from Makefile to .clang-tidy David Gibson
2024-11-05 23:25 ` [PATCH 04/12] arch: Avoid explicit access to 'environ' David Gibson
2024-11-05 23:25 ` [PATCH 05/12] flow: Correct type of flowside_at_sidx() David Gibson
2024-11-05 23:25 ` [PATCH 06/12] netlink: RTA_PAYLOAD() returns int, not size_t David Gibson
2024-11-05 23:25 ` [PATCH 07/12] Makefile: Move NETNS_RUN_DIR definition to C code David Gibson
2024-11-05 23:25 ` [PATCH 08/12] seccomp: Simplify handling of AUDIT_ARCH David Gibson
2024-11-05 23:25 ` [PATCH 09/12] Makefile: Use -DARCH for qrap only David Gibson
2024-11-05 23:25 ` [PATCH 10/12] Makefile: Don't attempt to auto-detect stack size David Gibson
2024-11-05 23:25 ` [PATCH 11/12] clang: Add rudimentary clangd configuration David Gibson
2024-11-05 23:25 ` [PATCH 12/12] util: Remove unused ffsl() function David Gibson
2024-11-06 19:13 ` [PATCH 00/12] Minor fixups for or inspired by clangd and related tools Stefano Brivio
2024-11-06 20:47   ` David Gibson
2024-11-07  7:03     ` Stefano Brivio
2024-11-07 14:55 ` Stefano Brivio

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241105232528.1408144-1-david@gibson.dropbear.id.au \
    --to=david@gibson.dropbear.id.au \
    --cc=passt-dev@passt.top \
    --cc=sbrivio@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).