Re: [PATCH 2/3] kconfig and lxdialog, kernel 2.6.13.4

From: Roman Zippel
Date: Thu Oct 27 2005 - 16:20:33 EST


Hi,

On Thu, 27 Oct 2005, Fao, Sean wrote:

BTW your patch is white space damaged and wordwrapped.

> @@ -92,14 +96,22 @@ int dialog_yesno(const char *title, cons
> case 'n':
> delwin(dialog);
> return 1;
> -
> + case 'A':
> + case 'a':
> + if (btn_t == YESNOABORT)
> + {
> + delwin(dialog);
> + return 2;
> + }
> + break;

Actually it's already possible to abort the dialog by pressing Esc. I
don't mind the abort button, but please match the Esc behaviour and return
-1.

bye, Roman
-
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/