On 25 Mar 2025, at 16:11, Aaron Conole wrote:

> Checkpatch has the ability to scan source files rather than exclusively
> scanning patch files.  In this mode, some checks get disabled (ex: all
> of the header and commit message parsing) and the display modes are
> adjusted to simply print the file and line, rather than including the patch
> line details.
>
> When checkpatch was updated for subject parsing, it was inadvertently
> ignoring the source file mode.  This means any checks run against a source
> file will always print a bogus subject suggestion, and warning.  Fix this
> by not printing these warnings when in source file mode.  This should have
> no effect on checkpatch against actual patch files.
>
> Fixes: 1b8fa4a66aa4 ("checkpatch: Add checks for the subject line.")
> Signed-off-by: Aaron Conole <[email protected]>

Did some testing, and the change looks good to me.

Acked-by: Eelco Chaudron <[email protected]>

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to