antimicrox icon indicating copy to clipboard operation
antimicrox copied to clipboard

Investigate warnings from QCommandLineParser (caused by some paritaly implemented features)

Open pktiuk opened this issue 2 years ago • 2 comments

After running antimicrox we get warnings from QCommandLineParser informing about checking not defined options.
These are caused by some partialy implemented features

antimicrox --log-level info
❗WARN  QCommandLineParser: option not defined: "next"
❗WARN  QCommandLineParser: option not defined: "map"
❗WARN  QCommandLineParser: option not defined: "display"

Code adding them to commandline is commented. I don't know why.

        {"display",
            QCoreApplication::translate("main", "Use specified display for
            X11 calls")},
        {"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")},

TODO

Investigate these features and fully enable (finish/fix) them or remove them from code.

pktiuk avatar Aug 18 '21 12:08 pktiuk

I think it would be good to just remove these flags with linked code.
These functions are too specific, and nobody was missing them.

pktiuk avatar Jan 20 '22 20:01 pktiuk

Use case of the --next flag was mentioned in this discussion: https://github.com/AntiMicroX/antimicrox/discussions/774

pktiuk avatar Aug 26 '23 20:08 pktiuk