laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/27203 )
Change subject: lint: checkpatch: show hint about typedefs_osmo.txt
......................................................................
lint: checkpatch: show hint about typedefs_osmo.txt
Change-Id: Icaee700a9cc7896cdbda6cce93db41b6b50a7fa3
---
M lint/checkpatch/checkpatch.pl
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/lint/checkpatch/checkpatch.pl b/lint/checkpatch/checkpatch.pl
index 51c406e..d98d79c 100755
--- a/lint/checkpatch/checkpatch.pl
+++ b/lint/checkpatch/checkpatch.pl
@@ -5151,7 +5151,7 @@
}
} elsif ($ctx =~ /Wx[^WCE]|[^WCE]xW/) {
if (ERROR("SPACING",
- "need consistent spacing around '$op' $at\n" . $hereptr)) {
+ "need consistent spacing around '$op' $at (or typedef missing in
osmo-ci/lint/checkpatch/typedefs_osmo.txt?)\n" . $hereptr)) {
$good = rtrim($fix_elements[$n]) . " " . trim($fix_elements[$n + 1]) .
" ";
if (defined $fix_elements[$n + 2]) {
$fix_elements[$n + 2] =~ s/^\s+//;
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ci/+/27203
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Icaee700a9cc7896cdbda6cce93db41b6b50a7fa3
Gerrit-Change-Number: 27203
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged