<div dir="auto">I found a bunch more cases will send V2</div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Feb 17, 2025, 12:24 Morten Brørup <<a href="mailto:mb@smartsharesystems.com">mb@smartsharesystems.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> From: Stephen Hemminger [mailto:<a href="mailto:stephen@networkplumber.org" target="_blank" rel="noreferrer">stephen@networkplumber.org</a>]<br>
> Sent: Monday, 17 February 2025 19.54<br>
> <br>
> A common pattern in DPDK internals is to have an array of function<br>
> pointers. When using these pointers an unnecessary derefence is done;<br>
> this is allowed in C because of a historical quirk but unnecessary.<br>
> <br>
> The PVS Studio checker flags these as:<br>
> <a href="https://pvs-studio.com/en/docs/warnings/v516/" rel="noreferrer noreferrer" target="_blank">https://pvs-studio.com/en/docs/warnings/v516/</a><br>
> <br>
> This leads to a large number of warnings marked as High severity.<br>
> This patchset simplifies the source and gets rid of these warnings.<br>
> <br>
> It should have no impact on the resulting binary.<br>
<br>
+1 to this RFC.<br>
The code becomes much cleaner to read.<br>
<br>
</blockquote></div>