* [PATCH] kconfig: qconf: Fix find on split mode
@ 2020-06-28 12:04 Mauro Carvalho Chehab
0 siblings, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2020-06-28 12:04 UTC (permalink / raw)
To: Linux Media Mailing List
Cc: Mauro Carvalho Chehab, Masahiro Yamada, linux-kbuild,
linux-kernel, Maxim Levitsky
The logic handling find on split mode is currently broken.
Fix it, making it work again as expected.
Reported-by: Maxim Levitsky <mlevitsk@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
scripts/kconfig/qconf.cc | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/scripts/kconfig/qconf.cc b/scripts/kconfig/qconf.cc
index c0ac8f7b5f1a..b8f577c6e8aa 100644
--- a/scripts/kconfig/qconf.cc
+++ b/scripts/kconfig/qconf.cc
@@ -1645,22 +1645,21 @@ void ConfigMainWindow::setMenuLink(struct menu *menu)
return;
list->setRootMenu(parent);
break;
- case symbolMode:
+ case menuMode:
if (menu->flags & MENU_ROOT) {
- configList->setRootMenu(menu);
+ menuList->setRootMenu(menu);
configList->clearSelection();
- list = menuList;
- } else {
list = configList;
+ } else {
+ configList->setRootMenu(menu);
+ configList->clearSelection();
+
parent = menu_get_parent_menu(menu->parent);
if (!parent)
return;
- item = menuList->findConfigItem(parent);
- if (item) {
- item->setSelected(true);
- menuList->scrollToItem(item);
- }
- list->setRootMenu(parent);
+ menuList->setRootMenu(parent);
+
+ list = menuList;
}
break;
case fullMode:
--
2.26.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] kconfig: qconf: Fix find on split mode
2020-06-28 12:18 Markus Elfring
@ 2020-06-28 12:26 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2020-06-28 12:26 UTC (permalink / raw)
To: Markus Elfring
Cc: linux-kbuild, linux-media, linux-kernel, kernel-janitors,
Masahiro Yamada, Maxim Levitsky
Em Sun, 28 Jun 2020 14:18:22 +0200
Markus Elfring <Markus.Elfring@web.de> escreveu:
> > The logic handling find on split mode is currently broken.
>
> * Is there a word missing in this change description?
>
> * Can any information become clearer another bit?
>
>
> > Fix it, …
>
> Please replace the beginning of this sentence with the tag “Fixes”.
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=719fdd32921fb7e3208db8832d32ae1c2d68900f#n183
>
> Regards,
> Markus
Please ignore this one. I ended re-submitting a previously
merged patch.
Thanks,
Mauro
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] kconfig: qconf: Fix find on split mode
@ 2020-06-28 12:18 Markus Elfring
2020-06-28 12:26 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 3+ messages in thread
From: Markus Elfring @ 2020-06-28 12:18 UTC (permalink / raw)
To: Mauro Carvalho Chehab, linux-kbuild, linux-media
Cc: linux-kernel, kernel-janitors, Masahiro Yamada, Maxim Levitsky
> The logic handling find on split mode is currently broken.
* Is there a word missing in this change description?
* Can any information become clearer another bit?
> Fix it, …
Please replace the beginning of this sentence with the tag “Fixes”.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=719fdd32921fb7e3208db8832d32ae1c2d68900f#n183
Regards,
Markus
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-06-28 12:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-28 12:04 [PATCH] kconfig: qconf: Fix find on split mode Mauro Carvalho Chehab
2020-06-28 12:18 Markus Elfring
2020-06-28 12:26 ` Mauro Carvalho Chehab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).