From: Sam Ravnborg Date: Sat, 19 Nov 2005 21:17:55 +0000 (+0100) Subject: kconfig: lxdialog is now sparse clean X-Git-Tag: upstream/snapshot3+hdmi~43593^2~16^2~29^2~24 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a06104af7dcf2f5bafaf18f373c8b2554cbfe014;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git kconfig: lxdialog is now sparse clean Replacing a gcc idiom with malloc and deleting an unused global variable made lxdialog sparse clean. Signed-off-by: Sam Ravnborg --- diff --git a/scripts/lxdialog/menubox.c b/scripts/lxdialog/menubox.c index 260cc4d..ff3a617 100644 --- a/scripts/lxdialog/menubox.c +++ b/scripts/lxdialog/menubox.c @@ -67,7 +67,7 @@ static void print_item(WINDOW * win, const char *item, int choice, int selected, int hotkey) { int j; - char menu_item[menu_width + 1]; + char *menu_item = malloc(menu_width + 1); strncpy(menu_item, item, menu_width); menu_item[menu_width] = 0; @@ -95,6 +95,7 @@ static void print_item(WINDOW * win, const char *item, int choice, wmove(win, choice, item_x + 1); wrefresh(win); } + free(menu_item); } /* @@ -221,7 +222,7 @@ int dialog_menu(const char *title, const char *prompt, int height, int width, /* * Find length of longest item in order to center menu. - * Set 'choice' to default item. + * Set 'choice' to default item. */ item_x = 0; for (i = 0; i < item_no; i++) { diff --git a/scripts/lxdialog/util.c b/scripts/lxdialog/util.c index 1f84809..ce41147 100644 --- a/scripts/lxdialog/util.c +++ b/scripts/lxdialog/util.c @@ -26,8 +26,6 @@ bool use_colors = 1; const char *backtitle = NULL; -const char *dialog_result; - /* * Attribute values, default is for mono display */