[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