tempo-plotter Changeset - 1c69ffd0aa9d · protofusion repositories

Changeset - 1c69ffd0aa9d

Parent rev.

Child rev.

[Not reviewed]

default

0 4 0

Hasan Yavuz ÖZDERYA - 8 years ago 2017-04-30 12:04:27
hy@ozderya.net

refactor symbol setting saving

4 files changed with 61 insertions and 58 deletions:

src/plot.cpp

src/plot.h

src/plotmanager.cpp

src/plotmanager.h

0 comments (0 inline, 0 general)

src/plot.cpp

➞

Show inline comments

@@ @@ -39,7 +39,7 @@ Plot::Plot(QWidget* parent) : @@
     isAutoScaled = true;
     symbolSize = 0;
     numOfSamples = 1;
-    showSymbols = Plot::Auto;
+    showSymbols = Plot::ShowSymbolsAuto;
     QObject::connect(&zoomer, &Zoomer::unzoomed, this, &Plot::unzoomed);
@@ @@ -252,11 +252,11 @@ void Plot::setSymbols(ShowSymbols shown) @@
+{
     showSymbols = shown;
-    if (showSymbols == Plot::Auto)
+    if (showSymbols == Plot::ShowSymbolsAuto)
+    {
         calcSymbolSize();
+    }
-    else if (showSymbols == Plot::Show)
+    else if (showSymbols == Plot::ShowSymbolsShow)
+    {
         symbolSize = SYMBOL_SIZE_MAX;
+    }
@@ @@ -271,7 +271,7 @@ void Plot::setSymbols(ShowSymbols shown) @@
 void Plot::onXScaleChanged()
+{
-    if (showSymbols == Plot::Auto)
+    if (showSymbols == Plot::ShowSymbolsAuto)
+    {
         calcSymbolSize();
         updateSymbols();

src/plot.h

➞

Show inline comments

@@ @@ -40,14 +40,12 @@ class Plot : public QwtPlot @@
     friend class PlotManager;
 public:
     // Note: this naming is also used as setting values
     enum ShowSymbols
+    {
         Auto,
         Show,
         Hide
         ShowSymbolsAuto,
         ShowSymbolsShow,
         ShowSymbolsHide
     };
     Q_ENUM(ShowSymbols);
     Plot(QWidget* parent = 0);
     ~Plot();

src/plotmanager.cpp

➞

Show inline comments

@@ @@ -44,7 +44,7 @@ PlotManager::PlotManager(QWidget* plotAr @@
     isDemoShown = false;
     _infoModel = infoModel;
     _numOfSamples = 1;
-    showSymbols = Plot::Auto;
+    showSymbols = Plot::ShowSymbolsAuto;
     // initalize layout and single widget
     isMulti = false;
@@ @@ -79,35 +79,35 @@ PlotManager::PlotManager(QWidget* plotAr @@
     showMinorGridAction.setEnabled(false);
     // setup symbols menu
     setSymbolAutoAct = setSymbolsMenu.addAction("Show When Zoomed");
     setSymbolAutoAct->setCheckable(true);
     setSymbolAutoAct->setChecked(true);
     connect(setSymbolAutoAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
     setSymbolsAutoAct = setSymbolsMenu.addAction("Show When Zoomed");
     setSymbolsAutoAct->setCheckable(true);
     setSymbolsAutoAct->setChecked(true);
     connect(setSymbolsAutoAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
             [this](bool checked)
+            {
-                if (checked) setSymbols(Plot::Auto);
+                if (checked) setSymbols(Plot::ShowSymbolsAuto);
             });
     setSymbolAlwaysAct = setSymbolsMenu.addAction("Always Show");
     setSymbolAlwaysAct->setCheckable(true);
     connect(setSymbolAlwaysAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
     setSymbolsShowAct = setSymbolsMenu.addAction("Always Show");
     setSymbolsShowAct->setCheckable(true);
     connect(setSymbolsShowAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
             [this](bool checked)
+            {
-                if (checked) setSymbols(Plot::Show);
+                if (checked) setSymbols(Plot::ShowSymbolsShow);
             });
     setSymbolHideAct = setSymbolsMenu.addAction("Always Hide");
     setSymbolHideAct->setCheckable(true);
     connect(setSymbolHideAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
     setSymbolsHideAct = setSymbolsMenu.addAction("Always Hide");
     setSymbolsHideAct->setCheckable(true);
     connect(setSymbolsHideAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
             [this](bool checked)
+            {
-                if (checked) setSymbols(Plot::Hide);
+                if (checked) setSymbols(Plot::ShowSymbolsHide);
             });
     setSymbolsAction.setMenu(&setSymbolsMenu);
     // add symbol actions to same group so that they appear as radio buttons
     auto group = new QActionGroup(this);
     group->addAction(setSymbolAutoAct);
     group->addAction(setSymbolAlwaysAct);
     group->addAction(setSymbolHideAct);
     group->addAction(setSymbolsAutoAct);
     group->addAction(setSymbolsShowAct);
     group->addAction(setSymbolsHideAct);
     connect(&showGridAction, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
             this, &PlotManager::showGrid);
@@ @@ -526,8 +526,22 @@ void PlotManager::saveSettings(QSettings @@
     settings->setValue(SG_Plot_MinorGrid, showMinorGridAction.isChecked());
     settings->setValue(SG_Plot_Legend, showLegendAction.isChecked());
     settings->setValue(SG_Plot_MultiPlot, showMultiAction.isChecked());
     settings->setValue(SG_Plot_Symbols,
                        QMetaEnum::fromType<Plot::ShowSymbols>().valueToKey(showSymbols));
     QString showSymbolsStr;
     if (showSymbols == Plot::ShowSymbolsAuto)
+    {
         showSymbolsStr = "auto";
+    }
     else if (showSymbols == Plot::ShowSymbolsShow)
+    {
         showSymbolsStr = "show";
+    }
     else
+    {
         showSymbolsStr = "hide";
+    }
     settings->setValue(SG_Plot_Symbols, showSymbolsStr);
     settings->endGroup();
+}
@@ @@ -551,34 +565,25 @@ void PlotManager::loadSettings(QSettings @@
         settings->value(SG_Plot_MultiPlot, showMultiAction.isChecked()).toBool());
     setMulti(showMultiAction.isChecked());
     // symbol setting
     QString showSymbolsStr = settings->value(SG_Plot_Symbols, QString()).toString();
     if (showSymbolsStr == "auto")
+    {
         setSymbols(Plot::ShowSymbolsAuto);
         setSymbolsAutoAct->setChecked(true);
+    }
     else if (showSymbolsStr == "show")
+    {
         bool ok = false;
         QString curStr = QMetaEnum::fromType<Plot::ShowSymbols>().valueToKey(showSymbols);
         QString settStr = settings->value(SG_Plot_Symbols, curStr).toString();
         Plot::ShowSymbols settValue =
             (Plot::ShowSymbols) QMetaEnum::fromType<Plot::ShowSymbols>().\
             keyToValue(settStr.toLatin1().constData(), &ok);
         if (ok)
+        {
             if (settValue == Plot::Auto)
+            {
                 setSymbolAutoAct->setChecked(true);
+            }
             else if (settValue == Plot::Show)
+            {
                 setSymbolAlwaysAct->setChecked(true);
+            }
             else
+            {
                 setSymbolHideAct->setChecked(true);
+            }
             setSymbols(settValue);
+        }
         else
+        {
             qCritical() << "Invalid symbol setting:" << settStr;
+        }
         setSymbols(Plot::ShowSymbolsShow);
         setSymbolsShowAct->setChecked(true);
+    }
     else if (showSymbolsStr == "hide")
+    {
         setSymbols(Plot::ShowSymbolsHide);
         setSymbolsHideAct->setChecked(true);
+    }
     else
+    {
         qCritical() << "Invalid symbol setting:" << showSymbolsStr;
+    }
     settings->endGroup();

src/plotmanager.h

➞

Show inline comments

@@ @@ -102,9 +102,9 @@ private: @@
     QAction showMultiAction;
     QAction setSymbolsAction;
     QMenu setSymbolsMenu;
     QAction* setSymbolAutoAct;
     QAction* setSymbolAlwaysAct;
     QAction* setSymbolHideAct;
     QAction* setSymbolsAutoAct;
     QAction* setSymbolsShowAct;
     QAction* setSymbolsHideAct;
     void setupLayout(bool multiPlot);
     /// Inserts a new plot widget to the current layout.

0 comments (0 inline, 0 general)