On 18 Oct 2023, at 12:46, Simon Horman wrote:
> On Wed, Oct 18, 2023 at 12:39:51PM +0200, Simon Horman wrote:
>> On Tue, Oct 17, 2023 at 03:49:31PM +0200, Eelco Chaudron wrote:
>>> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
>>> ---
>>> lib/meta-flow.c | 4 ++--
>>> lib/ofp-actions.c | 8 +++++---
>>> 2 files changed, 7 insertions(+), 5 deletions(-)
>>
>> Thanks Eelco,
>>
>> I agree that these initialisations are unnecessary.
>> Although I am unsure how to get clang to emit warnings about them.
>
> Of course shortly after writing the above I figured out that
> I can use something like:
>
> $ clang-tidy lib/meta-flow.c -- -I. -I./include
Never used clang-tidy, but looks interesting ;)
I use the following from the testing document:
$ ./configure CC=clang
$ make clang-analyze
It does give you a nice flow on how clang came to the conclusion.
//Eelco
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
