tempo-plotter Changeset - 6862f9405cdb · protofusion repositories

Changeset - 6862f9405cdb

Parent rev.

Child rev.

[Not reviewed]

feature/command-line-options

0 3 0

Mehmet Aslan - 6 years ago 2019-04-04 03:23:29
aaslan-mehmet@hotmail.com

Nonexisting configuration file log changed and handled.
Minor name changes and fixes.

3 files changed with 13 insertions and 11 deletions:

CMakeLists.txt

src/main.cpp

src/mainwindow.cpp

0 comments (0 inline, 0 general)

CMakeLists.txt

➞

Show inline comments

@@ @@ -175,13 +175,13 @@ include(GetVersion) @@
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DVERSION_STRING=\\\"${VERSION_STRING}\\\" ")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DVERSION_MAJOR=${VERSION_MAJOR} ")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DVERSION_MINOR=${VERSION_MINOR} ")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DVERSION_PATCH=${VERSION_PATCH} ")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DVERSION_REVISION=\\\"${VERSION_REVISION}\\\" ")
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DPROGRAM_NAME_STRING=\\\"${PROGRAM_NAME}\\\" ")
+set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DPROGRAM_NAME=\\\"${PROGRAM_NAME}\\\" ")
 # add make run target
 add_custom_target(run
     COMMAND ${PROGRAM_NAME}
     DEPENDS ${PROGRAM_NAME}
     WORKING_DIRECTORY ${CMAKE_PROJECT_DIR}

src/main.cpp

➞

Show inline comments

@@ @@ -67,13 +67,13 @@ void messageHandler(QtMsgType type, cons @@
+    }
+}
 int main(int argc, char *argv[])
+{
     QApplication a(argc, argv);
-    QApplication::setApplicationName(PROGRAM_NAME_STRING);
+    QApplication::setApplicationName(PROGRAM_NAME);
     QApplication::setApplicationVersion(VERSION_STRING);
     qInstallMessageHandler(messageHandler);
     MainWindow w;
     pMainWindow = &w;

src/mainwindow.cpp

➞

Show inline comments

@@ @@ -32,12 +32,13 @@ @@
 #include <QCommandLineParser>
 #include <QFileInfo>
 #include <qwt_plot.h>
 #include <limits.h>
 #include <cmath>
 #include <iostream>
 #include <cstdlib>
 #include <plot.h>
 #include <barplot.h>
 #include "framebufferseries.h"
 #include "utils.h"
@@ @@ -250,15 +251,17 @@ MainWindow::MainWindow(QWidget *parent) @@
     // init stream connections
     connect(&dataFormatPanel, &DataFormatPanel::sourceChanged,
             this, &MainWindow::onSourceChanged);
     onSourceChanged(dataFormatPanel.activeSource());
     // load default settings
-    QSettings settings(PROGRAM_NAME_STRING, PROGRAM_NAME_STRING);
+    QSettings settings(PROGRAM_NAME, PROGRAM_NAME);
     loadAllSettings(&settings);
     handleCommandLineOptions(*QApplication::instance());
     // ensure command panel has 1 command if none loaded
     if (!commandPanel.numOfCommands())
+    {
         commandPanel.newCommandAction()->trigger();
+    }
@@ @@ -267,14 +270,12 @@ MainWindow::MainWindow(QWidget *parent) @@
     // the very first run.
     connect(commandPanel.newCommandAction(), &QAction::triggered, [this]()
+            {
                 this->ui->tabWidget->setCurrentWidget(&commandPanel);
                 this->ui->tabWidget->showTabs();
             });
     handleCommandLineOptions(*QApplication::instance());
+}
 MainWindow::~MainWindow()
+{
     if (serialPort.isOpen())
+    {
@@ @@ -301,13 +302,13 @@ void MainWindow::closeEvent(QCloseEvent @@
             event->ignore();
             return;
+        }
+    }
     // save settings
-    QSettings settings(PROGRAM_NAME_STRING, PROGRAM_NAME_STRING);
+    QSettings settings(PROGRAM_NAME, PROGRAM_NAME);
     saveAllSettings(&settings);
     settings.sync();
     if (settings.status() != QSettings::NoError)
+    {
         QString errorText;
@@ @@ -598,37 +599,38 @@ void MainWindow::handleCommandLineOption @@
     QCommandLineParser parser;
     parser.setSingleDashWordOptionMode(QCommandLineParser::ParseAsCompactedShortOptions);
     parser.setApplicationDescription("Small and simple software for plotting data from serial port in realtime.");
     parser.addHelpOption();
     parser.addVersionOption();
-    QCommandLineOption loadOpt({"l", "load"}, "Load settings from file.", "filename");
+    QCommandLineOption configOpt({"c", "config"}, "Load configuration from file.", "filename");
     QCommandLineOption portOpt({"p", "port"}, "Set port name.", "port name");
     QCommandLineOption baudrateOpt({"b" ,"baudrate"}, "Set port baud rate.", "baud rate");
     QCommandLineOption openPortOpt({"o", "open"}, "Open serial port.");
-    parser.addOption(loadOpt);
+    parser.addOption(configOpt);
     parser.addOption(portOpt);
     parser.addOption(baudrateOpt);
     parser.addOption(openPortOpt);
     parser.process(app);
-    if (parser.isSet(loadOpt))
+    if (parser.isSet(configOpt))
+    {
-        QString fileName = parser.value(loadOpt);
+        QString fileName = parser.value(configOpt);
         QFileInfo fileInfo(fileName);
         if (fileInfo.exists() && fileInfo.isFile())
+        {
             QSettings settings(fileName, QSettings::IniFormat);
             loadAllSettings(&settings);
+        }
         else
+        {
             qWarning() << "Ini file not exist";
             qCritical() << "Configuration file not exist. Closing application.";
             std::exit(1);
+        }
+    }
     if (parser.isSet(portOpt))
+    {
         portControl.selectPort(parser.value(portOpt));

0 comments (0 inline, 0 general)