On Sun, Jun 25, 2023 at 5:36 AM Jesse T <mr.bossman075@xxxxxxxxx> wrote: > > On Sat, Jun 24, 2023 at 4:11 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > > > On Sun, Jun 18, 2023 at 9:16 AM Jesse Taube <mr.bossman075@xxxxxxxxx> wrote: > > > > > > Menuconfig has a feature where you can "press the key in the (#) prefix > > > to jump directly to that location. You will be returned to the current > > > search results after exiting this new menu." > > > > > > This commit adds this feature to nconfig, with almost identical code. > > > > > > Signed-off-by: Jesse Taube <Mr.Bossman075@xxxxxxxxx> > > > > > > Setting the code duplication aside, > > > If it does function as expected will you accept the patch? Probably. The menuconfig code you copied is really ugly. Actually, I was refactoring the search jump code a few months ago, but I did not get around to completing it. > Should I mark the copied codes' original location? > > > does this patch work correctly? > > > > > > > > > > $ make defconfig > > $ make nconfig > > > > Press F8 > > > > Input "MODULES" in the search box. > > > > You will see: > > > > (1) -> Enable loadable module support > > > > Press the "1" key. > > > > > > It will navigate to "General setup" > > instead of "Enable loadable module support". > > > Hmm, this is a confusing issue. > It will take you to the parent menu of the option, it should move you > to the parent option and > move the cursor to the option location. I will see if I can resolve > this oversight. > > Thanks, > Jesse Taube > > > > > > > > > > > > > > > > > -- > > Best Regards > > Masahiro Yamada -- Best Regards Masahiro Yamada
