Remove unused --map flag used for opening only mapping window

Author: pktiuk

src/antimicrosettings.cpp

@@ -63,10 +63,6 @@ void AntiMicroSettings::importFromCommandLine(CommandLineUtility &cmdutility)
     {
         getCmdSettings().setValue("LaunchInTray", 1);
     }
-    if (cmdutility.shouldMapController())
-    {
-        getCmdSettings().setValue("DisplaySDLMapping", 1);
-    }
 }
 /**
  * @brief applies settings from command line to logger

src/commandlineutility.cpp

@@ -41,7 +41,6 @@ CommandLineUtility::CommandLineUtility(QObject *parent)
     unloadProfile = false;
     startSetNumber = 0;
     listControllers = false;
-    mappingController = false;
     currentLogLevel = Logger::LOG_NONE;
 
     currentListsIndex = 0;
@@ -104,11 +103,7 @@ void CommandLineUtility::parseArguments(const QApplication &parsed_app)
         // {"next",
         //     QCoreApplication::translate("main", "Advance profile loading set
         //     options")},
-        //  {"map",
-        //      QCoreApplication::translate("main", "Open game controller
-        //      mapping window of selected controller. Value can be
-        //      a controller index or GUID."),
-        //      QCoreApplication::translate("main", "value")},
+
     });
 
     parser.process(parsed_app);
@@ -173,11 +168,6 @@ void CommandLineUtility::parseArguments(const QApplication &parsed_app)
             listControllers = true;
         }
 
-        if (parser.isSet("map"))
-        {
-            parseArgsMap(parser);
-        }
-
 #if (defined(WITH_UINPUT) && defined(WITH_XTEST))
 
         if (parser.isSet("eventgen"))
@@ -380,33 +370,6 @@ void CommandLineUtility::parseArgsStartSet(const QCommandLineParser &parser)
     }
 }
 
-void CommandLineUtility::parseArgsMap(const QCommandLineParser &parser)
-{
-    QString mapOptionText = parser.value("map");
-
-    if (!mapOptionText.isEmpty())
-    {
-        bool validNumber = false;
-        int tempNumber = mapOptionText.toInt(&validNumber);
-
-        if (validNumber)
-        {
-            controllerNumber = tempNumber;
-            mappingController = true;
-        } else if (!mapOptionText.isEmpty())
-        {
-            controllerIDString = mapOptionText;
-            mappingController = true;
-        } else
-        {
-            throw std::runtime_error(QObject::tr("Controller identifier is not a valid value.").toStdString());
-        }
-    } else
-    {
-        throw std::runtime_error(QObject::tr("No controller was specified.").toStdString());
-    }
-}
-
 bool CommandLineUtility::isLaunchInTrayEnabled() { return launchInTray; }
 
 bool CommandLineUtility::isTrayHidden() { return hideTrayIcon; }
@@ -435,8 +398,6 @@ int CommandLineUtility::getJoyStartSetNumber() { return startSetNumber - 1; }
 
 bool CommandLineUtility::shouldListControllers() { return listControllers; }
 
-bool CommandLineUtility::shouldMapController() { return mappingController; }
-
 QString CommandLineUtility::getEventGenerator() { return eventGenerator; }
 
 Logger::LogLevel CommandLineUtility::getCurrentLogLevel() { return currentLogLevel; }

src/commandlineutility.h

@@ -102,7 +102,6 @@ class CommandLineUtility : public QObject
     bool isShowRequested();
     bool isUnloadRequested();
     bool shouldListControllers();
-    bool shouldMapController();
     bool hasProfileInOptions();
 
     int getControllerNumber();
@@ -127,7 +126,6 @@ class CommandLineUtility : public QObject
     bool showRequest;
     bool unloadProfile;
     bool listControllers;
-    bool mappingController;
 
     int startSetNumber;
     int controllerNumber;
@@ -148,7 +146,6 @@ class CommandLineUtility : public QObject
     void parseArgsPrControle(const QCommandLineParser &parser);
     void parseArgsUnload(const QCommandLineParser &parser);
     void parseArgsStartSet(const QCommandLineParser &parser);
-    void parseArgsMap(const QCommandLineParser &parser);
 };
 
 #endif // COMMANDLINEPARSER_H

src/gui/gamecontrollermappingdialog.cpp

@@ -332,14 +332,8 @@ void GameControllerMappingDialog::saveChanges()
     settings->setValue(QString("Mappings/%1%2").arg(device->getUniqueIDString()).arg("Disable"), "0");
     settings->sync();
 
-    bool displayMapping = settings->runtimeValue("DisplaySDLMapping", false).toBool();
     settings->getLock()->unlock();
 
-    if (displayMapping)
-    {
-        PRINT_STDOUT() << generateSDLMappingString();
-    }
-
     emit mappingUpdate(mappingString, device);
 }
 

src/gui/mainwindow.cpp

@@ -120,8 +120,7 @@ MainWindow::MainWindow(QMap<SDL_JoystickID, InputDevice *> *joysticks, CommandLi
 #endif
 
     signalDisconnect = false;
-    showTrayIcon = !cmdutility->isTrayHidden() && graphical && !cmdutility->shouldListControllers() &&
-                   !cmdutility->shouldMapController();
+    showTrayIcon = !cmdutility->isTrayHidden() && graphical && !cmdutility->shouldListControllers();
 
     m_joysticks = joysticks;
 
@@ -174,7 +173,6 @@ MainWindow::MainWindow(QMap<SDL_JoystickID, InputDevice *> *joysticks, CommandLi
     connect(ui->actionOptions, &QAction::triggered, this, &MainWindow::openMainSettingsDialog);
     connect(ui->actionWiki, &QAction::triggered, this, &MainWindow::openWikiPage);
     connect(ui->actionCalibration, &QAction::triggered, this, &MainWindow::openCalibration);
-    connect(ui->actionGameController_Mapping, &QAction::triggered, this, &MainWindow::openGameControllerMappingWindow);
 
 #if defined(WITH_X11)
     if (QApplication::platformName() == QStringLiteral("xcb"))
@@ -299,44 +297,6 @@ void MainWindow::alterConfigFromSettings()
     }
 }
 
-void MainWindow::controllerMapOpening()
-{
-    if (m_cmdutility->shouldMapController())
-    {
-        m_graphical = false;
-
-        const QList<ControllerOptionsInfo> tempList = m_cmdutility->getControllerOptionsList();
-        ControllerOptionsInfo temp = tempList.at(0);
-
-        if (temp.hasControllerNumber())
-        {
-            int joypadIndex = m_cmdutility->getControllerNumber();
-
-            qDebug() << "It was antimicrox --map controllerNumber";
-            qDebug() << "controllerNumber: " << joypadIndex;
-
-            selectControllerJoyTab(joypadIndex);
-            openGameControllerMappingWindow(true);
-        } else if (temp.hasControllerID())
-        {
-            QString joypadGUID = m_cmdutility->getControllerID();
-
-            qDebug() << "It was antimicrox --map controllerID";
-            qDebug() << "controllerID: " << joypadGUID;
-
-            selectControllerJoyTab(joypadGUID);
-            openGameControllerMappingWindow(true);
-        } else
-        {
-            qDebug() << "Could not find a proper controller identifier. Exiting";
-
-            qInfo() << tr("Could not find a proper controller identifier. "
-                          "Exiting.");
-            qApp->quit();
-        }
-    }
-}
-
 void MainWindow::fillButtons() { fillButtonsMap(m_joysticks); }
 
 void MainWindow::makeJoystickTabs()
@@ -1468,36 +1428,6 @@ void MainWindow::restartAsElevated()
 }
 #endif
 
-void MainWindow::openGameControllerMappingWindow(bool openAsMain)
-{
-    int index = ui->tabWidget->currentIndex();
-    if (index >= 0)
-    {
-        JoyTabWidget *joyTab = qobject_cast<JoyTabWidget *>(ui->tabWidget->widget(index)); // static_cast
-        InputDevice *joystick = joyTab->getJoystick();
-        if (joystick != nullptr)
-        {
-            GameControllerMappingDialog *dialog = new GameControllerMappingDialog(joystick, m_settings, this);
-
-            if (openAsMain)
-            {
-                dialog->setParent(nullptr);
-                dialog->setWindowFlags(Qt::Window);
-                connect(dialog, &GameControllerMappingDialog::finished, qApp, &QApplication::quit);
-            } else
-            {
-                connect(dialog, &GameControllerMappingDialog::mappingUpdate, this, &MainWindow::propogateMappingUpdate);
-            }
-
-            dialog->show();
-        }
-    } else if (openAsMain)
-    {
-        qInfo() << tr("Could not find controller. Exiting.");
-        qApp->quit();
-    }
-}
-
 void MainWindow::propogateMappingUpdate(QString mapping, InputDevice *device) { emit mappingUpdated(mapping, device); }
 
 void MainWindow::testMappingUpdateNow(int index, InputDevice *device)
@@ -1806,26 +1736,6 @@ void MainWindow::updateButtonPressed()
 
 #endif
 
-/**
- * @brief Select appropriate tab with the specified index.
- * @param Index of appropriate tab.
- */
-void MainWindow::selectControllerJoyTab(int index)
-{
-    if ((index > 0) && m_joysticks->contains(index - 1))
-    {
-        JoyTabWidget *widget = qobject_cast<JoyTabWidget *>(ui->tabWidget->widget(index - 1)); // static_cast
-        if (widget != nullptr)
-        {
-            qDebug() << "JoyTabWidget was not a null pointer in selectControllerJoyTab of index";
-            ui->tabWidget->setCurrentIndex(index - 1);
-        } else
-        {
-            qDebug() << "JoyTabWidget was a NULL POINTER in selectControllerJoyTab of index";
-        }
-    }
-}
-
 /**
  * @brief Select appropriate tab that has a device with the specified GUID.
  * @param GUID of joystick device.

src/gui/mainwindow.h

@@ -107,12 +107,10 @@ class MainWindow : public QMainWindow
     void quitProgram();
     void changeWindowStatus();
     void refreshTabHelperThreads();
-    void controllerMapOpening();
     void testMappingUpdateNow(int index, InputDevice *device);
     void removeJoyTab(SDL_JoystickID deviceID);
     void addJoyTab(InputDevice *device);
     void selectControllerJoyTab(QString GUID);
-    void selectControllerJoyTab(int index);
     void handleInstanceDisconnect();
 
   private slots:
@@ -144,7 +142,6 @@ class MainWindow : public QMainWindow
     void restartAsElevated();
 #endif
 
-    void openGameControllerMappingWindow(bool openAsMain = false);
     void propogateMappingUpdate(QString mapping, InputDevice *device);
     void autoprofileLoad(AutoProfileInfo *info); // MainConfiguration class
     void checkAutoProfileWatcherTimer();         // MainConfiguration class

src/main.cpp

@@ -479,67 +479,6 @@ int main(int argc, char *argv[])
 #endif
         delete appLogger;
         return 0;
-    } else if (cmdutility.shouldMapController())
-    {
-        PadderCommon::mouseHelperObj.initDeskWid();
-        QPointer<InputDaemon> joypad_worker = new InputDaemon(joysticks, &settings);
-        inputEventThread = new QThread;
-
-        MainWindow *mainWindow = new MainWindow(joysticks, &cmdutility, &settings);
-
-        QObject::connect(&antimicrox, &QApplication::aboutToQuit, mainWindow, &MainWindow::removeJoyTabs);
-        QObject::connect(&antimicrox, &QApplication::aboutToQuit, joypad_worker.data(), &InputDaemon::quit);
-        QObject::connect(&antimicrox, &QApplication::aboutToQuit, &PadderCommon::mouseHelperObj, &MouseHelper::deleteDeskWid,
-                         Qt::DirectConnection);
-        QObject::connect(&antimicrox, &QApplication::aboutToQuit, joypad_worker.data(), &InputDaemon::deleteLater,
-                         Qt::BlockingQueuedConnection);
-
-        mainWindow->makeJoystickTabs();
-#if QT_VERSION >= QT_VERSION_CHECK(5, 4, 0)
-        QTimer::singleShot(0, mainWindow, &MainWindow::controllerMapOpening);
-#else
-        QTimer::singleShot(0, mainWindow, SLOT(controllerMapOpening()));
-#endif
-
-        joypad_worker->startWorker();
-
-        joypad_worker->moveToThread(inputEventThread);
-        PadderCommon::mouseHelperObj.moveToThread(inputEventThread);
-        inputEventThread->start(QThread::HighPriority);
-
-        int app_result = antimicrox.exec();
-
-        inputEventThread->quit();
-        inputEventThread->wait();
-
-        delete joysticks;
-        joysticks = nullptr;
-
-        delete localServer;
-        localServer = nullptr;
-
-        delete inputEventThread;
-        inputEventThread = nullptr;
-
-#ifdef WITH_X11
-
-        if (QApplication::platformName() == QStringLiteral("xcb"))
-        {
-            X11Extras::getInstance()->closeDisplay();
-        }
-
-#endif
-
-        delete mainWindow;
-        mainWindow = nullptr;
-
-        if (!joypad_worker.isNull())
-        {
-            delete joypad_worker;
-            joypad_worker.clear();
-        }
-        delete appLogger;
-        return app_result;
     }
 
     bool status = true;