summaryrefslogtreecommitdiff
path: root/extra/config/menu.c
diff options
context:
space:
mode:
authorEric Andersen <andersen@codepoet.org>2004-07-15 05:54:13 +0000
committerEric Andersen <andersen@codepoet.org>2004-07-15 05:54:13 +0000
commit6c2efd66d94ce13ac5b70aefd71cf2e66f4acbdf (patch)
tree4614629205be8798da130da69d369387d093ce38 /extra/config/menu.c
parentdb5a9ef35b66e495feb4daf2e9576883dd52926f (diff)
Peter Kjellerstedt at axis.com writes:
Hello, the attached patch should bring extra/config in line with the Linux 2.6.7 sources. The following are the commit messages for the respective files from the Linux bk-repository: checklist.c: * fix menuconfig choice item help display confdata.c: * config: choice fix * kconfig: don't rename target dir when saving config expr.c, expr.h: * config: disable debug prints mconf.c: * fix menuconfig choice item help display menu.c: * Kconfig: use select statements symbol.c: * config: choice fix * Avoid bogus warning about recursive dependencies * c99 struct initialiser conversions textbox.c: * janitor: don't init statics to 0 util.c: * fix lxdialog behaviour //Peter
Diffstat (limited to 'extra/config/menu.c')
-rw-r--r--extra/config/menu.c125
1 files changed, 80 insertions, 45 deletions
diff --git a/extra/config/menu.c b/extra/config/menu.c
index ba5c6a6c5..bcc4f098b 100644
--- a/extra/config/menu.c
+++ b/extra/config/menu.c
@@ -16,6 +16,26 @@ static struct menu **last_entry_ptr;
struct file *file_list;
struct file *current_file;
+static void menu_warn(struct menu *menu, const char *fmt, ...)
+{
+ va_list ap;
+ va_start(ap, fmt);
+ fprintf(stderr, "%s:%d:warning: ", menu->file->name, menu->lineno);
+ vfprintf(stderr, fmt, ap);
+ fprintf(stderr, "\n");
+ va_end(ap);
+}
+
+static void prop_warn(struct property *prop, const char *fmt, ...)
+{
+ va_list ap;
+ va_start(ap, fmt);
+ fprintf(stderr, "%s:%d:warning: ", prop->file->name, prop->lineno);
+ vfprintf(stderr, fmt, ap);
+ fprintf(stderr, "\n");
+ va_end(ap);
+}
+
void menu_init(void)
{
current_entry = current_menu = &rootmenu;
@@ -94,9 +114,9 @@ void menu_set_type(int type)
sym->type = type;
return;
}
- fprintf(stderr, "%s:%d:warning: type of '%s' redefined from '%s' to '%s'\n",
- current_entry->file->name, current_entry->lineno,
- sym->name ? sym->name : "<choice>", sym_type_name(sym->type), sym_type_name(type));
+ menu_warn(current_entry, "type of '%s' redefined from '%s' to '%s'\n",
+ sym->name ? sym->name : "<choice>",
+ sym_type_name(sym->type), sym_type_name(type));
}
struct property *menu_add_prop(enum prop_type type, char *prompt, struct expr *expr, struct expr *dep)
@@ -110,8 +130,7 @@ struct property *menu_add_prop(enum prop_type type, char *prompt, struct expr *e
if (prompt) {
if (current_entry->prompt)
- fprintf(stderr, "%s:%d: prompt redefined\n",
- current_entry->file->name, current_entry->lineno);
+ menu_warn(current_entry, "prompt redefined\n");
current_entry->prompt = prop;
}
@@ -133,6 +152,50 @@ void menu_add_symbol(enum prop_type type, struct symbol *sym, struct expr *dep)
menu_add_prop(type, NULL, expr_alloc_symbol(sym), dep);
}
+void sym_check_prop(struct symbol *sym)
+{
+ struct property *prop;
+ struct symbol *sym2;
+ for (prop = sym->prop; prop; prop = prop->next) {
+ switch (prop->type) {
+ case P_DEFAULT:
+ if ((sym->type == S_STRING || sym->type == S_INT || sym->type == S_HEX) &&
+ prop->expr->type != E_SYMBOL)
+ prop_warn(prop,
+ "default for config symbol '%'"
+ " must be a single symbol", sym->name);
+ break;
+ case P_SELECT:
+ sym2 = prop_get_symbol(prop);
+ if (sym->type != S_BOOLEAN && sym->type != S_TRISTATE)
+ prop_warn(prop,
+ "config symbol '%s' uses select, but is "
+ "not boolean or tristate", sym->name);
+ else if (sym2->type == S_UNKNOWN)
+ prop_warn(prop,
+ "'select' used by config symbol '%s' "
+ "refer to undefined symbol '%s'",
+ sym->name, sym2->name);
+ else if (sym2->type != S_BOOLEAN && sym2->type != S_TRISTATE)
+ prop_warn(prop,
+ "'%s' has wrong type. 'select' only "
+ "accept arguments of boolean and "
+ "tristate type", sym2->name);
+ break;
+ case P_RANGE:
+ if (sym->type != S_INT && sym->type != S_HEX)
+ prop_warn(prop, "range is only allowed "
+ "for int or hex symbols");
+ if (!sym_string_valid(sym, prop->expr->left.sym->name) ||
+ !sym_string_valid(sym, prop->expr->right.sym->name))
+ prop_warn(prop, "range is invalid");
+ break;
+ default:
+ ;
+ }
+ }
+}
+
void menu_finalize(struct menu *parent)
{
struct menu *menu, *last_menu;
@@ -222,17 +285,16 @@ void menu_finalize(struct menu *parent)
if (sym && sym_is_choice(sym) && menu->sym) {
menu->sym->flags |= SYMBOL_CHOICEVAL;
if (!menu->prompt)
- fprintf(stderr, "%s:%d:warning: choice value must have a prompt\n",
- menu->file->name, menu->lineno);
+ menu_warn(menu, "choice value must have a prompt");
for (prop = menu->sym->prop; prop; prop = prop->next) {
if (prop->type == P_PROMPT && prop->menu != menu) {
- fprintf(stderr, "%s:%d:warning: choice values currently only support a single prompt\n",
- prop->file->name, prop->lineno);
-
+ prop_warn(prop, "choice values "
+ "currently only support a "
+ "single prompt");
}
if (prop->type == P_DEFAULT)
- fprintf(stderr, "%s:%d:warning: defaults for choice values not supported\n",
- prop->file->name, prop->lineno);
+ prop_warn(prop, "defaults for choice "
+ "values not supported");
}
current_entry = menu;
menu_set_type(sym->type);
@@ -256,42 +318,15 @@ void menu_finalize(struct menu *parent)
}
if (sym && !(sym->flags & SYMBOL_WARNED)) {
- struct symbol *sym2;
if (sym->type == S_UNKNOWN)
- fprintf(stderr, "%s:%d:warning: config symbol defined without type\n",
- parent->file->name, parent->lineno);
+ menu_warn(parent, "config symbol defined "
+ "without type\n");
if (sym_is_choice(sym) && !parent->prompt)
- fprintf(stderr, "%s:%d:warning: choice must have a prompt\n",
- parent->file->name, parent->lineno);
-
- for (prop = sym->prop; prop; prop = prop->next) {
- switch (prop->type) {
- case P_DEFAULT:
- if ((sym->type == S_STRING || sym->type == S_INT || sym->type == S_HEX) &&
- prop->expr->type != E_SYMBOL)
- fprintf(stderr, "%s:%d:warning: default must be a single symbol\n",
- prop->file->name, prop->lineno);
- break;
- case P_SELECT:
- sym2 = prop_get_symbol(prop);
- if ((sym->type != S_BOOLEAN) || (sym2->type != S_BOOLEAN))
- fprintf(stderr, "%s:%d:warning: enable is only allowed with booleans\n",
- prop->file->name, prop->lineno);
- break;
- case P_RANGE:
- if (sym->type != S_INT && sym->type != S_HEX)
- fprintf(stderr, "%s:%d:warning: range is only allowed for int or hex symbols\n",
- prop->file->name, prop->lineno);
- if (!sym_string_valid(sym, prop->expr->left.sym->name) ||
- !sym_string_valid(sym, prop->expr->right.sym->name))
- fprintf(stderr, "%s:%d:warning: range is invalid\n",
- prop->file->name, prop->lineno);
- break;
- default:
- ;
- }
- }
+ menu_warn(parent, "choice must have a prompt\n");
+
+ /* Check properties connected to this symbol */
+ sym_check_prop(sym);
sym->flags |= SYMBOL_WARNED;
}