<feed xmlns='http://www.w3.org/2005/Atom'>
<title>passt/.clang-tidy, branch master</title>
<subtitle>Plug A Simple Socket Transport</subtitle>
<link rel='alternate' type='text/html' href='https://passt.top/passt/'/>
<entry>
<title>clang-tidy: Don't insist on #ifdef over #if defined()</title>
<updated>2026-03-04T16:53:33+00:00</updated>
<author>
<name>David Gibson</name>
<email>david@gibson.dropbear.id.au</email>
</author>
<published>2026-03-02T04:31:35+00:00</published>
<link rel='alternate' type='text/html' href='https://passt.top/passt/commit/?id=685864dea8c35a6abf3ab61de786167e73b18816'/>
<id>685864dea8c35a6abf3ab61de786167e73b18816</id>
<content type='text'>
Commit 036fb8770 ("checksum: add VSX fast path for POWER8/POWER9") changed
an #ifdef to #if defined, in order to match a newly introduced #if which
needs to check two different symbols.  This causes clang-tidy to complain
that the directive could be written more concisely.

In this case consistency with the other #if branch seems more important,
and in general insisting on #ifdef over #if seems unhelpfully pedantic.
Suppress that warning globally.

Reported-by: Laurent Vivier &lt;lvivier@redhat.com&gt;
Fixes: 036fb8770cc2 ("checksum: add VSX fast path for POWER8/POWER9")
Signed-off-by: David Gibson &lt;david@gibson.dropbear.id.au&gt;
Signed-off-by: Stefano Brivio &lt;sbrivio@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Commit 036fb8770 ("checksum: add VSX fast path for POWER8/POWER9") changed
an #ifdef to #if defined, in order to match a newly introduced #if which
needs to check two different symbols.  This causes clang-tidy to complain
that the directive could be written more concisely.

In this case consistency with the other #if branch seems more important,
and in general insisting on #ifdef over #if seems unhelpfully pedantic.
Suppress that warning globally.

Reported-by: Laurent Vivier &lt;lvivier@redhat.com&gt;
Fixes: 036fb8770cc2 ("checksum: add VSX fast path for POWER8/POWER9")
Signed-off-by: David Gibson &lt;david@gibson.dropbear.id.au&gt;
Signed-off-by: Stefano Brivio &lt;sbrivio@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>clang: Move clang-tidy configuration from Makefile to .clang-tidy</title>
<updated>2024-11-07T11:46:19+00:00</updated>
<author>
<name>David Gibson</name>
<email>david@gibson.dropbear.id.au</email>
</author>
<published>2024-11-05T23:25:19+00:00</published>
<link rel='alternate' type='text/html' href='https://passt.top/passt/commit/?id=b78e72da0b27e222592ff1f1578c69bad4756c65'/>
<id>b78e72da0b27e222592ff1f1578c69bad4756c65</id>
<content type='text'>
Currently we configure clang-tidy with a very long command line spelled out
in the Makefile (mostly a big list of lints to disable).  Move it from here
into a .clang-tidy configuration file, so that the config is accessible if
clang-tidy is invoked in other ways (e.g. via clangd) as well.  As a bonus
this also means that we can move the bulky comments about why we're
suppressing various tests inline with the relevant config lines.

Signed-off-by: David Gibson &lt;david@gibson.dropbear.id.au&gt;
Signed-off-by: Stefano Brivio &lt;sbrivio@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Currently we configure clang-tidy with a very long command line spelled out
in the Makefile (mostly a big list of lints to disable).  Move it from here
into a .clang-tidy configuration file, so that the config is accessible if
clang-tidy is invoked in other ways (e.g. via clangd) as well.  As a bonus
this also means that we can move the bulky comments about why we're
suppressing various tests inline with the relevant config lines.

Signed-off-by: David Gibson &lt;david@gibson.dropbear.id.au&gt;
Signed-off-by: Stefano Brivio &lt;sbrivio@redhat.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
