[coreboot-gerrit] New patch to review for coreboot: kconfig_lint: Don't look at IS_ENABLED() text in comments.
Martin Roth (martinroth@google.com)
gerrit at coreboot.org
Tue Jan 26 04:17:52 CET 2016
Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13458
-gerrit
commit fac977babdc0ae4dff102c010f0cea16fbd9b0bc
Author: Martin Roth <martinroth at google.com>
Date: Mon Jan 25 16:14:09 2016 -0700
kconfig_lint: Don't look at IS_ENABLED() text in comments.
This fixes at least one kconfig_lint warning.
Change-Id: I35edf57e90315a8372aaf3b41e923cd8dad7386a
Signed-off-by: Martin Roth <martinroth at google.com>
---
util/lint/kconfig_lint | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/util/lint/kconfig_lint b/util/lint/kconfig_lint
index 44e898f..2cb125e 100755
--- a/util/lint/kconfig_lint
+++ b/util/lint/kconfig_lint
@@ -253,7 +253,7 @@ sub check_is_enabled {
my $file = $1;
my $lineno = $2;
$line = $3;
- if ( $line !~ /(.*)IS_ENABLED\s*\(\s*CONFIG_(\w+)(.*)/ ) {
+ if ( ( $line !~ /(.*)IS_ENABLED\s*\(\s*CONFIG_(\w+)(.*)/ ) && ( $line !~ /(\/[\*\/])(.*)IS_ENABLED/ ) ) {
show_warning("# uninterpreted IS_ENABLED at $file:$lineno: $line");
next;
}
More information about the coreboot-gerrit
mailing list