Re: [PATCH] kconfig: qconf: make debug links work again

From: Masahiro Yamada
Date: Sun Jun 28 2020 - 10:59:38 EST


On Sun, Jun 28, 2020 at 11:48 PM Maxim Levitsky <mlevitsk@xxxxxxxxxx> wrote:
>
> On Sun, 2020-06-28 at 23:41 +0900, Masahiro Yamada wrote:
> > On Sun, Jun 28, 2020 at 9:21 PM Mauro Carvalho Chehab
> > <mchehab+huawei@xxxxxxxxxx> wrote:
> > > The Qt5 conversion broke support for debug info links.
> > >
> > > Restore the behaviour added by changeset
> > > ab45d190fd4a ("kconfig: create links in info window").
> > >
> > > Reported-by: Maxim Levitsky <mlevitsk@xxxxxxxxxx>
> > > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
> >
> > I tested this patch, but this caused
> > segmentation fault.
> >
> >
> > I enabled 'Show Debug Info',
> > and then clicked
> > dep: <symbol name>.
> >
> > Then, xconfig crashed.
> >
> > (without this patch, it did not cause
> > segfault at least)
> >
> > Did you see this?
>
> Works for me - tested this again
> (I have both patches applied on top on mainline master branch).
> Maybe you have Qt4?


I do not think so.

I checked scripts/kconfig/.qconf.cmd

qconf was linked with Qt5.


$ cat scripts/kconfig/.qconf.cmd
cmd_scripts/kconfig/qconf := g++ -o scripts/kconfig/qconf
scripts/kconfig/images.o scripts/kconfig/confdata.o
scripts/kconfig/expr.o scripts/kconfig/lexer.lex.o
scripts/kconfig/parser.tab.o scripts/kconfig/preprocess.o
scripts/kconfig/symbol.o scripts/kconfig/util.o
scripts/kconfig/qconf.o -lQt5Widgets -lQt5Gui -lQt5Core


BTW, my machine runs ubuntu 20.04




>
> One thing that I forgot to report is that when clicking on the symbol,
> only config descripion updates and not config/menu windows.
> It might even be always like that, I don't remember, but it would be nice if
> these were updated too.
>
> Best regards,
> Maxim Levitsky
>
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> > > ---
> > > scripts/kconfig/qconf.cc | 35 ++++++++++++++++++++++++++++++++++-
> > > scripts/kconfig/qconf.h | 1 +
> > > 2 files changed, 35 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/scripts/kconfig/qconf.cc b/scripts/kconfig/qconf.cc
> > > index 631e19659504..03cadf27a376 100644
> > > --- a/scripts/kconfig/qconf.cc
> > > +++ b/scripts/kconfig/qconf.cc
> > > @@ -1012,7 +1012,7 @@ ConfigInfoView::ConfigInfoView(QWidget* parent, const char *name)
> > > : Parent(parent), sym(0), _menu(0)
> > > {
> > > setObjectName(name);
> > > -
> > > + setOpenLinks(false);
> > >
> > > if (!objectName().isEmpty()) {
> > > configSettings->beginGroup(objectName());
> > > @@ -1224,6 +1224,36 @@ void ConfigInfoView::expr_print_help(void *data, struct symbol *sym, const char
> > > *text += str2;
> > > }
> > >
> > > +void ConfigInfoView::clicked(const QUrl &url)
> > > +{
> > > + QByteArray str = url.toEncoded();
> > > + const std::size_t count = str.size();
> > > + char *hex = new char[count];
> > > + unsigned long p;
> > > +
> > > + if (count < 1)
> > > + return;
> > > +
> > > + memcpy(hex, str.constData(), count);
> > > + p = (int)strtol(hex + 1, NULL, 16);
> > > +
> > > + if (!p)
> > > + return;
> > > +
> > > + if (hex[0] == 's') {
> > > + struct symbol *s = (struct symbol *)p;
> > > +
> > > + sym = s;
> > > + symbolInfo();
> > > + } else {
> > > + struct menu *m = (struct menu *)p;
> > > +
> > > + _menu = m;
> > > + menuInfo();
> > > + }
> > > + emit showDebugChanged(true);
> > > +}
> > > +
> > > QMenu* ConfigInfoView::createStandardContextMenu(const QPoint & pos)
> > > {
> > > QMenu* popup = Parent::createStandardContextMenu(pos);
> > > @@ -1497,6 +1527,9 @@ ConfigMainWindow::ConfigMainWindow(void)
> > > helpMenu->addAction(showIntroAction);
> > > helpMenu->addAction(showAboutAction);
> > >
> > > + connect (helpText, SIGNAL (anchorClicked (const QUrl &)),
> > > + helpText, SLOT (clicked (const QUrl &)) );
> > > +
> > > connect(configList, SIGNAL(menuChanged(struct menu *)),
> > > helpText, SLOT(setInfo(struct menu *)));
> > > connect(configList, SIGNAL(menuSelected(struct menu *)),
> > > diff --git a/scripts/kconfig/qconf.h b/scripts/kconfig/qconf.h
> > > index d913a02967ae..a193137f2314 100644
> > > --- a/scripts/kconfig/qconf.h
> > > +++ b/scripts/kconfig/qconf.h
> > > @@ -250,6 +250,7 @@ public slots:
> > > void setInfo(struct menu *menu);
> > > void saveSettings(void);
> > > void setShowDebug(bool);
> > > + void clicked (const QUrl &url);
> > >
> > > signals:
> > > void showDebugChanged(bool);
> > > --
> > > 2.26.2
> > >
> >
> >
>
>


--
Best Regards
Masahiro Yamada