Re: [PATCH] [kconfig] Direct use of lxdialog routines by menuconfig (resent,v2)

• Next message: Maciej Soltysiak: "Is this patch okay?"
• Previous message: Matthew Garrett: "Re: #! incompatible -- binfmt_script.c broken?"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Sat Nov 23, 2002 at 10:50:40AM +0100, Petr Baudis wrote:
> if (menu == &rootmenu) {
> - cprint(":");
> - cprint("--- ");
> - cprint("L");
> - cprint("Load an Alternate Configuration File");
> - cprint("S");
> - cprint("Save Configuration to an Alternate File");
> - }
> - stat = exec_conf();
> + cmake(); cprint_tag(":"); cprint_name("--- ");
> + cmake(); cprint_tag("L"); cprint_name("Load an Alternate Configuration File");
> + cmake(); cprint_tag("S"); cprint_name("Save Configuration to an Alternate File");
> + }
> + dialog_clear();
> + stat = dialog_menu(prompt ? prompt : "Main Menu",
> + menu_instructions, rows, cols, rows - 10,
> + active_entry, item_no, items);
> if (stat < 0)
> continue;

You never return from sub-menus. This last bit should be:

                  if (stat < 0)
- continue;
+ return;

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

• Next message: Maciej Soltysiak: "Is this patch okay?"
• Previous message: Matthew Garrett: "Re: #! incompatible -- binfmt_script.c broken?"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Sat Dec 07 2002 - 22:00:20 EST