Re: [PATCH 0/6] menuconfig: jump to search results

[Borislav Petkov]

On Mon, Jul 30, 2012 at 04:22:12PM -0400, Benjamin Poirier wrote:
> It's against linux-next, which already contains some patches for
> menuconfig.

Ok, applying against the linux-next from today succeeds. And they seem
to work, I can search for CONFIG_* stuff and then go straight to the
results.

A couple more observations:

* The numbering in the search results looks kinda strange and
non-intuitive IMHO. Would it be possible to use the ncurses cursor to
iterate over the search results like one does in the normal menus? I.e.,
the horizontal bar which highlights the item with the background color
or whatever - can it be moved up and down with the arrow keys through
the search results too?

* when searching for "DELAY" for example, I get results without numbers
to which I can't jump to:

...
  â Symbol: DEFAULT_IO_DELAY_TYPE [=0]
  â Type  : integer
  â
  â
  â Symbol: IO_DELAY_TYPE_0X80 [=0]
  â Type  : integer
...

maybe because they don't have "---help---" section?

In those cases, you probably want to hide those from the search results?

Other than that it looks good and I'd say we need more menuconfig
functionality like that which eases/speeds up working with that thing :).

Thanks.

-- 
Regards/Gruss,
    Boris.
--
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/