kconfig: return 'false' instead of 'no' in bool function
authorVegard Nossum <vegard.nossum@oracle.com>
Fri, 1 Jan 2016 16:34:05 +0000 (17:34 +0100)
committerMichal Marek <mmarek@suse.com>
Tue, 5 Jan 2016 10:19:13 +0000 (11:19 +0100)
menu_is_visible() is a bool function and should use boolean return
values. "no" is a tristate value which happens to also have a value
of 0, but we should nevertheless use the right symbol for it.

This is a very minor cleanup with no semantic change.

Fixes: 86e187ff9 ("kconfig: add an option to determine a menu's visibility")
Cc: Arnaud Lacombe <lacombar@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
Signed-off-by: Vegard Nossum <vegard.nossum@oracle.com>
Signed-off-by: Michal Marek <mmarek@suse.com>
scripts/kconfig/menu.c

index b05cc3d..aed678e 100644 (file)
@@ -477,7 +477,7 @@ bool menu_is_visible(struct menu *menu)
 
        if (menu->visibility) {
                if (expr_calc_value(menu->visibility) == no)
-                       return no;
+                       return false;
        }
 
        sym = menu->sym;