Re: [PATCH] nconfig: add search support
From: Randy Dunlap
Date: Thu Aug 05 2010 - 13:04:34 EST
On Tue, 3 Aug 2010 15:28:12 +0300 nir.tzachar@xxxxxxxxx wrote:
> From: Nir Tzachar <nir.tzachar@xxxxxxxxx>
> Remove the old hotkeys feature, and replace by a regular string search.
> Behaviour of search is as advised by Sam.
> From nconfig help:
> Searching: pressing '/' triggers search mode. nconfig performs a
> regular string compare, case insensitive, starting at
I would say: simple string compare
"regular" has no meaning (at least for me) there.
> the beginning of each menu line.\n"
> Pressing the up/down keys highlights the previous/next
> matching item. Backspace removes one character from the
> match string. Pressing either '/' again or ESC exits\
> search mode. All other keys behave normally.
> Miscellaneous other changes (including Rundy's Justin's input).
Yes, the bottom menu and the typos are fixed. Thanks.
However, I'm not finding this terribly useful, mostly due to it starting
comparison at the beginning of each menu line.
E.g., I'd like to be able to find "emulations" in the top-level menu here:
Executable file formats / Emulations --->
or I'd like to be able to find "crc" menu items in any sub-menu.
Maybe I should just stick to config symbol searches. I don't think it's all
that likely that people will know how each menu line text begins.
As for the search UI, I'd rather that it be presented like the symbol search,
in a box, instead of just a single line at the top of the screen.
> Signed-off-by: Nir Tzachar <nir.tzachar@xxxxxxxxx>
> scripts/kconfig/nconf.c | 350 +++++++++++++++++++++++++------------------
> scripts/kconfig/nconf.gui.c | 20 ++--
> scripts/kconfig/nconf.h | 3 +-
> 3 files changed, 213 insertions(+), 160 deletions(-)
*** Remember to use Documentation/SubmitChecklist when testing your code ***
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/