Re: [PATCH] nconfig: add search support

[Randy Dunlap <rdunlap@xenotime.net>]

On Sun,  1 Aug 2010 10:20:33 +0300 nir.tzachar@gmail.com wrote:

> From: Nir Tzachar <nir.tzachar@gmail.com>
> 
> Remove the old hotkeys feature, and replace by a regular string search.
> From nconfig help:
> 
> Pressing '/' triggers search mode. nconfig does regular string match, case
> insensitive, starting at the begining of each menu line.

What is a "regular string match"?
That's not the same as a regular expression match, right?

Maybe it's a "simple string match"?

> Pressing Enter highlights the next match, Backspace removes one character from
> the match string. Pressing either '/' again or ESC exits search mode.
> 
> Rebind the '/' key (which allowed to search for symbols) to F8
> 
> Signed-off-by: Nir Tzachar <nir.tzachar@gmail.com>
> ---
>  scripts/kconfig/nconf.c |  261 ++++++++++++++++++++++-------------------------
>  scripts/kconfig/nconf.h |    3 +-
>  2 files changed, 125 insertions(+), 139 deletions(-)
> 
> diff --git a/scripts/kconfig/nconf.c b/scripts/kconfig/nconf.c
> index 762caf8..2a7cb37 100644
> --- a/scripts/kconfig/nconf.c
> +++ b/scripts/kconfig/nconf.c
> @@ -41,9 +41,12 @@ static const char nconf_readme[] = N_(
>  "   pressing <Enter> of <right-arrow>. Use <Esc> or <left-arrow> to go back.\n"
>  "   Submenus are designated by \"--->\".\n"
>  "\n"
> -"   Shortcut: Press the option's highlighted letter (hotkey).\n"
> -"             Pressing a hotkey more than once will sequence\n"
> -"             through all visible items which use that hotkey.\n"
> +"   Searching: pressing '/' triggers search mode. nconfig does a\n"
> +"              regular string match, case insensitive, starting at\n"
> +"              the begining of each menu line.\n"
> +"              Pressing Enter highlights the next match, Backspace\n"
> +"              removes one character from the match string.\n"
> +"              Pressing either '/' again or ESC exits search mode.\n"
>  "\n"
>  "   You may also use the <PAGE UP> and <PAGE DOWN> keys to scroll\n"
>  "   unseen options into view.\n"


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***