Re: [PATCH 0/2] nconfig

From: Nir Tzachar
Date: Sat Jan 09 2010 - 11:47:31 EST


On Thu, Jan 7, 2010 at 4:52 PM, Michal Marek <mmarek@xxxxxxx> wrote:
> Hi Nir,
>
> Just FYI: I queued nconfig for the next merge window. I added a patch to
> mark some functions static and changed the description of your patch,
> otherwise it's the v7 version you posted.

Thats gr8. Thanks.

I would also appreciate if you can add the patch I sent Jan (attached
below). It fixed the problem Jan raised. If you need, I'll make it a
proper patch on top of v7.

Cheers.

diff --git a/scripts/kconfig/nconf.c b/scripts/kconfig/nconf.c
index 8c56150..cab6bb0 100644
--- a/scripts/kconfig/nconf.c
+++ b/scripts/kconfig/nconf.c
@@ -984,7 +984,7 @@ static void build_conf(struct menu *menu)
break;
default:
tmp = 2 + strlen(sym_get_string_value(sym));
- item_make(menu, 's', "(%s)",
+ item_make(menu, 's', " (%s)",
sym_get_string_value(sym));
tmp = indent - tmp + 4;
if (tmp < 0)
@@ -1072,8 +1072,8 @@ static void show_menu(const char *prompt, const
char *instructions,

/* position the menu at the middle of the screen */
scale_menu(curses_menu, &maxy, &maxx);
- maxx = min(maxx, mwin_max_cols);
- maxy = mwin_max_lines-1;
+ maxx = min(maxx, mwin_max_cols-2);
+ maxy = mwin_max_lines-2;
menu_window = derwin(main_window,
maxy,
maxx,
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/