Re: [PATCH] netfilter: conntrack: fix -Wformat

From: Nick Desaulniers
Date: Wed Dec 02 2020 - 17:35:49 EST


On Tue, Nov 10, 2020 at 2:04 PM Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> On Tue, 2020-11-10 at 14:00 -0800, Nick Desaulniers wrote:
>
> > Yeah, we could go through and remove %h and %hh to solve this, too, right?
>
> Yup.
>
> I think one of the checkpatch improvement mentees is adding
> some suggestion and I hope an automated fix mechanism for that.
>
> https://lore.kernel.org/lkml/5e3265c241602bb54286fbaae9222070daa4768e.camel@xxxxxxxxxxx/

+ Tom, who's been looking at leveraging clang-tidy to automate such
treewide mechanical changes.
ex. https://reviews.llvm.org/D91789

See also commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouraging
use of unnecessary %h[xudi] and %hh[xudi]") for a concise summary of
related context.
--
Thanks,
~Nick Desaulniers