kconfig: qconf: do not use 'menu' variable for (QMenuBar *)
authorMasahiro Yamada <masahiroy@kernel.org>
Fri, 7 Aug 2020 09:18:52 +0000 (18:18 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Fri, 14 Aug 2020 04:30:04 +0000 (13:30 +0900)
I think it is a bit confusing to use 'menu' to hold a QMenuBar pointer.
I want to use 'menu' for a QMenu pointer.

You do not need to use a local variable here. Use menuBar() directly.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
scripts/kconfig/qconf.cc

index cf2e06ebc4a8248311dd732032fdcb0321f4996d..79275c44d7bef2f5fdfd5e414e425fbe3804d18b 100644 (file)
@@ -1382,7 +1382,6 @@ void ConfigSearchWindow::search(void)
 ConfigMainWindow::ConfigMainWindow(void)
        : searchWindow(0)
 {
-       QMenuBar* menu;
        bool ok = true;
        QVariant x, y;
        int width, height;
@@ -1433,7 +1432,6 @@ ConfigMainWindow::ConfigMainWindow(void)
        setTabOrder(configList, helpText);
        configList->setFocus();
 
-       menu = menuBar();
        toolBar = new QToolBar("Tools", this);
        addToolBar(toolBar);
 
@@ -1519,7 +1517,7 @@ ConfigMainWindow::ConfigMainWindow(void)
        toolBar->addAction(fullViewAction);
 
        // create config menu
-       QMenu* config = menu->addMenu("&File");
+       QMenu* config = menuBar()->addMenu("&File");
        config->addAction(loadAction);
        config->addAction(saveAction);
        config->addAction(saveAsAction);
@@ -1527,12 +1525,13 @@ ConfigMainWindow::ConfigMainWindow(void)
        config->addAction(quitAction);
 
        // create edit menu
-       QMenu* editMenu = menu->addMenu("&Edit");
+       QMenu* editMenu = menuBar()->addMenu("&Edit");
        editMenu->addAction(searchAction);
 
        // create options menu
-       QMenu* optionMenu = menu->addMenu("&Option");
+       QMenu* optionMenu = menuBar()->addMenu("&Option");
        optionMenu->addAction(showNameAction);
+
        optionMenu->addAction(showRangeAction);
        optionMenu->addAction(showDataAction);
        optionMenu->addSeparator();
@@ -1541,7 +1540,7 @@ ConfigMainWindow::ConfigMainWindow(void)
        optionMenu->addAction(showDebugAction);
 
        // create help menu
-       QMenu* helpMenu = menu->addMenu("&Help");
+       QMenu* helpMenu = menuBar()->addMenu("&Help");
        helpMenu->addAction(showIntroAction);
        helpMenu->addAction(showAboutAction);