scripts/checkpatch.pl: Ignore ObjC #import lines for operator spacing
authorPhil Dennis-Jordan <phil@philjordan.eu>
Thu, 24 Oct 2024 12:35:55 +0000 (14:35 +0200)
committerThomas Huth <thuth@redhat.com>
Fri, 8 Nov 2024 05:13:33 +0000 (06:13 +0100)
checkpatch.pl lints for spaces around operators including / (slash).
Code lines starting with #include are ignored, as slashes in those
represent path separators.

In Objective-C code, #import is often used in preference to #include,
as preprocessor-based multiple-#include defenses are considered
non-idiomatic in that language.

This change extends checkpatch.pl to treat #import lines in the same
way as #include, avoiding false positives for "missing" spaces
around path separators on those lines.

Signed-off-by: Phil Dennis-Jordan <phil@philjordan.eu>
Message-ID: <20241024123555.25861-1-phil@philjordan.eu>
Signed-off-by: Thomas Huth <thuth@redhat.com>
scripts/checkpatch.pl

index 1b21249c91e7e32c06aa5f28250d7803eda8e2ed..06d07e6c225c1eac39d6249a257d160429734caf 100755 (executable)
@@ -2246,7 +2246,7 @@ sub process {
                        }
                }
 # Check operator spacing.
-               if (!($line=~/\#\s*include/)) {
+               if (!($line=~/\#\s*(include|import)/)) {
                        my $ops = qr{
                                <<=|>>=|<=|>=|==|!=|
                                \+=|-=|\*=|\/=|%=|\^=|\|=|&=|