diff options
author | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2021-12-26 11:05:25 +0100 |
---|---|---|
committer | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2021-12-26 11:05:25 +0100 |
commit | 35bc4590ae75d7178061205d8e937bc4f8b5b09e (patch) | |
tree | 91152ba23d96a739d13938ed4463d2854a4a900f | |
parent | 01e245417e24fe6c9957b35850db6b1473f66e98 (diff) | |
download | rockbox-35bc4590ae75d7178061205d8e937bc4f8b5b09e.tar.gz rockbox-35bc4590ae75d7178061205d8e937bc4f8b5b09e.zip |
rbutil: Remove ProgressLogger interface class.
Since we're using signals to connect to the Logger we don't need an
interface class, and the only thing still used was the log level enum.
Put those values in a dedicated header instead.
Change-Id: Ic3aee14ba13b81e283b477d11b1c15a5c26ffeab
-rw-r--r-- | utils/rbutilqt/CMakeLists.txt | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/bootloaderinstallbase.h | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/bootloaderinstallfile.h | 1 | ||||
-rw-r--r-- | utils/rbutilqt/base/bootloaderinstallmi4.h | 1 | ||||
-rw-r--r-- | utils/rbutilqt/base/mspackutil.cpp | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/progressloglevels.h (renamed from utils/rbutilqt/base/progressloggerinterface.h) | 35 | ||||
-rw-r--r-- | utils/rbutilqt/base/talkfile.h | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/talkgenerator.h | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/uninstall.h | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/voicefile.h | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/zipinstaller.h | 2 | ||||
-rw-r--r-- | utils/rbutilqt/base/ziputil.cpp | 2 | ||||
-rw-r--r-- | utils/rbutilqt/progressloggergui.cpp | 6 | ||||
-rw-r--r-- | utils/rbutilqt/progressloggergui.h | 4 | ||||
-rw-r--r-- | utils/rbutilqt/rbutilqt.cpp | 2 |
15 files changed, 16 insertions, 51 deletions
diff --git a/utils/rbutilqt/CMakeLists.txt b/utils/rbutilqt/CMakeLists.txt index e7a36de369..dfa0350f1b 100644 --- a/utils/rbutilqt/CMakeLists.txt +++ b/utils/rbutilqt/CMakeLists.txt | |||
@@ -195,7 +195,7 @@ add_library(rbbase | |||
195 | ${CMAKE_CURRENT_LIST_DIR}/base/httpget.h | 195 | ${CMAKE_CURRENT_LIST_DIR}/base/httpget.h |
196 | ${CMAKE_CURRENT_LIST_DIR}/base/mspackutil.cpp | 196 | ${CMAKE_CURRENT_LIST_DIR}/base/mspackutil.cpp |
197 | ${CMAKE_CURRENT_LIST_DIR}/base/mspackutil.h | 197 | ${CMAKE_CURRENT_LIST_DIR}/base/mspackutil.h |
198 | ${CMAKE_CURRENT_LIST_DIR}/base/progressloggerinterface.h | 198 | ${CMAKE_CURRENT_LIST_DIR}/base/progressloglevels.h |
199 | ${CMAKE_CURRENT_LIST_DIR}/base/playerbuildinfo.cpp | 199 | ${CMAKE_CURRENT_LIST_DIR}/base/playerbuildinfo.cpp |
200 | ${CMAKE_CURRENT_LIST_DIR}/base/playerbuildinfo.h | 200 | ${CMAKE_CURRENT_LIST_DIR}/base/playerbuildinfo.h |
201 | ${CMAKE_CURRENT_LIST_DIR}/base/rbsettings.cpp | 201 | ${CMAKE_CURRENT_LIST_DIR}/base/rbsettings.cpp |
diff --git a/utils/rbutilqt/base/bootloaderinstallbase.h b/utils/rbutilqt/base/bootloaderinstallbase.h index 23aac4f92f..9a216a632c 100644 --- a/utils/rbutilqt/base/bootloaderinstallbase.h +++ b/utils/rbutilqt/base/bootloaderinstallbase.h | |||
@@ -20,7 +20,7 @@ | |||
20 | #define BOOTLOADERINSTALLBASE_H | 20 | #define BOOTLOADERINSTALLBASE_H |
21 | 21 | ||
22 | #include <QtCore> | 22 | #include <QtCore> |
23 | #include "progressloggerinterface.h" | 23 | #include "progressloglevels.h" |
24 | #include "httpget.h" | 24 | #include "httpget.h" |
25 | 25 | ||
26 | //! baseclass for all Bootloader installs | 26 | //! baseclass for all Bootloader installs |
diff --git a/utils/rbutilqt/base/bootloaderinstallfile.h b/utils/rbutilqt/base/bootloaderinstallfile.h index 9f9586279c..d529939c8b 100644 --- a/utils/rbutilqt/base/bootloaderinstallfile.h +++ b/utils/rbutilqt/base/bootloaderinstallfile.h | |||
@@ -20,7 +20,6 @@ | |||
20 | #define BOOTLOADERINSTALLFILE_H | 20 | #define BOOTLOADERINSTALLFILE_H |
21 | 21 | ||
22 | #include <QtCore> | 22 | #include <QtCore> |
23 | #include "progressloggerinterface.h" | ||
24 | #include "bootloaderinstallbase.h" | 23 | #include "bootloaderinstallbase.h" |
25 | 24 | ||
26 | //! install a bootloader by putting a single file on the player. | 25 | //! install a bootloader by putting a single file on the player. |
diff --git a/utils/rbutilqt/base/bootloaderinstallmi4.h b/utils/rbutilqt/base/bootloaderinstallmi4.h index c56669b99f..4ba878caac 100644 --- a/utils/rbutilqt/base/bootloaderinstallmi4.h +++ b/utils/rbutilqt/base/bootloaderinstallmi4.h | |||
@@ -20,7 +20,6 @@ | |||
20 | #define BOOTLOADERINSTALLMI4_H | 20 | #define BOOTLOADERINSTALLMI4_H |
21 | 21 | ||
22 | #include <QtCore> | 22 | #include <QtCore> |
23 | #include "progressloggerinterface.h" | ||
24 | #include "bootloaderinstallbase.h" | 23 | #include "bootloaderinstallbase.h" |
25 | 24 | ||
26 | 25 | ||
diff --git a/utils/rbutilqt/base/mspackutil.cpp b/utils/rbutilqt/base/mspackutil.cpp index b794272199..49c064e4d2 100644 --- a/utils/rbutilqt/base/mspackutil.cpp +++ b/utils/rbutilqt/base/mspackutil.cpp | |||
@@ -19,7 +19,7 @@ | |||
19 | #include <QtCore> | 19 | #include <QtCore> |
20 | #include "Logger.h" | 20 | #include "Logger.h" |
21 | #include "mspackutil.h" | 21 | #include "mspackutil.h" |
22 | #include "progressloggerinterface.h" | 22 | #include "progressloglevels.h" |
23 | 23 | ||
24 | MsPackUtil::MsPackUtil(QObject* parent) | 24 | MsPackUtil::MsPackUtil(QObject* parent) |
25 | :ArchiveUtil(parent) | 25 | :ArchiveUtil(parent) |
diff --git a/utils/rbutilqt/base/progressloggerinterface.h b/utils/rbutilqt/base/progressloglevels.h index ae4270ae6b..95d479ca29 100644 --- a/utils/rbutilqt/base/progressloggerinterface.h +++ b/utils/rbutilqt/base/progressloglevels.h | |||
@@ -18,43 +18,12 @@ | |||
18 | * | 18 | * |
19 | ****************************************************************************/ | 19 | ****************************************************************************/ |
20 | 20 | ||
21 | #ifndef PROGRESSLOGGERINTERFACE_H | 21 | #ifndef PROGRESSLOGLEVELS_H |
22 | #define PROGRESSLOGGERINTERFACE_H | 22 | #define PROGRESSLOGLEVELS_H |
23 | |||
24 | #include <QtCore> | ||
25 | 23 | ||
26 | enum { | 24 | enum { |
27 | LOGNOICON, LOGOK, LOGINFO, LOGWARNING, LOGERROR | 25 | LOGNOICON, LOGOK, LOGINFO, LOGWARNING, LOGERROR |
28 | }; | 26 | }; |
29 | 27 | ||
30 | |||
31 | |||
32 | class ProgressloggerInterface : public QObject | ||
33 | { | ||
34 | Q_OBJECT | ||
35 | |||
36 | public: | ||
37 | ProgressloggerInterface(QObject* parent) : QObject(parent) {} | ||
38 | |||
39 | virtual void setProgressValue(int value)=0; | ||
40 | virtual void setProgressMax(int max)=0; | ||
41 | virtual int getProgressMax()=0; | ||
42 | |||
43 | signals: | ||
44 | void aborted(); | ||
45 | |||
46 | |||
47 | public slots: | ||
48 | virtual void addItem(const QString &text, int flag)=0; //! add a string to the list, with icon | ||
49 | |||
50 | virtual void close()=0; | ||
51 | virtual void show()=0; | ||
52 | virtual void setRunning()=0; | ||
53 | virtual void setFinished()=0; | ||
54 | |||
55 | private: | ||
56 | |||
57 | }; | ||
58 | |||
59 | #endif | 28 | #endif |
60 | 29 | ||
diff --git a/utils/rbutilqt/base/talkfile.h b/utils/rbutilqt/base/talkfile.h index 11dd95ca14..014753da2a 100644 --- a/utils/rbutilqt/base/talkfile.h +++ b/utils/rbutilqt/base/talkfile.h | |||
@@ -23,7 +23,7 @@ | |||
23 | #define TALKFILE_H | 23 | #define TALKFILE_H |
24 | 24 | ||
25 | #include <QtCore> | 25 | #include <QtCore> |
26 | #include "progressloggerinterface.h" | 26 | #include "progressloglevels.h" |
27 | 27 | ||
28 | #include "talkgenerator.h" | 28 | #include "talkgenerator.h" |
29 | 29 | ||
diff --git a/utils/rbutilqt/base/talkgenerator.h b/utils/rbutilqt/base/talkgenerator.h index 3e2f9394fb..bb026190d8 100644 --- a/utils/rbutilqt/base/talkgenerator.h +++ b/utils/rbutilqt/base/talkgenerator.h | |||
@@ -23,7 +23,7 @@ | |||
23 | #define TALKGENERATOR_H | 23 | #define TALKGENERATOR_H |
24 | 24 | ||
25 | #include <QtCore> | 25 | #include <QtCore> |
26 | #include "progressloggerinterface.h" | 26 | #include "progressloglevels.h" |
27 | 27 | ||
28 | #include "encoderbase.h" | 28 | #include "encoderbase.h" |
29 | #include "ttsbase.h" | 29 | #include "ttsbase.h" |
diff --git a/utils/rbutilqt/base/uninstall.h b/utils/rbutilqt/base/uninstall.h index d8c9815117..3858b779fa 100644 --- a/utils/rbutilqt/base/uninstall.h +++ b/utils/rbutilqt/base/uninstall.h | |||
@@ -24,7 +24,7 @@ | |||
24 | 24 | ||
25 | #include <QtCore> | 25 | #include <QtCore> |
26 | 26 | ||
27 | #include "progressloggerinterface.h" | 27 | #include "progressloglevels.h" |
28 | 28 | ||
29 | 29 | ||
30 | class Uninstaller : public QObject | 30 | class Uninstaller : public QObject |
diff --git a/utils/rbutilqt/base/voicefile.h b/utils/rbutilqt/base/voicefile.h index d34535e700..c813910a69 100644 --- a/utils/rbutilqt/base/voicefile.h +++ b/utils/rbutilqt/base/voicefile.h | |||
@@ -23,7 +23,7 @@ | |||
23 | #define VOICEFILE_H | 23 | #define VOICEFILE_H |
24 | 24 | ||
25 | #include <QtCore> | 25 | #include <QtCore> |
26 | #include "progressloggerinterface.h" | 26 | #include "progressloglevels.h" |
27 | 27 | ||
28 | #include "httpget.h" | 28 | #include "httpget.h" |
29 | #include "voicefont.h" | 29 | #include "voicefont.h" |
diff --git a/utils/rbutilqt/base/zipinstaller.h b/utils/rbutilqt/base/zipinstaller.h index 59a0f785d9..35967af6b1 100644 --- a/utils/rbutilqt/base/zipinstaller.h +++ b/utils/rbutilqt/base/zipinstaller.h | |||
@@ -24,7 +24,7 @@ | |||
24 | 24 | ||
25 | #include <QtCore> | 25 | #include <QtCore> |
26 | 26 | ||
27 | #include "progressloggerinterface.h" | 27 | #include "progressloglevels.h" |
28 | #include "httpget.h" | 28 | #include "httpget.h" |
29 | #include "Logger.h" | 29 | #include "Logger.h" |
30 | 30 | ||
diff --git a/utils/rbutilqt/base/ziputil.cpp b/utils/rbutilqt/base/ziputil.cpp index 45119f7d99..3527e8e881 100644 --- a/utils/rbutilqt/base/ziputil.cpp +++ b/utils/rbutilqt/base/ziputil.cpp | |||
@@ -19,7 +19,7 @@ | |||
19 | #include <QtCore> | 19 | #include <QtCore> |
20 | #include <QDebug> | 20 | #include <QDebug> |
21 | #include "ziputil.h" | 21 | #include "ziputil.h" |
22 | #include "progressloggerinterface.h" | 22 | #include "progressloglevels.h" |
23 | #include "Logger.h" | 23 | #include "Logger.h" |
24 | 24 | ||
25 | #include "quazip/quazip.h" | 25 | #include "quazip/quazip.h" |
diff --git a/utils/rbutilqt/progressloggergui.cpp b/utils/rbutilqt/progressloggergui.cpp index 78fc00db8f..f04de395a3 100644 --- a/utils/rbutilqt/progressloggergui.cpp +++ b/utils/rbutilqt/progressloggergui.cpp | |||
@@ -22,7 +22,7 @@ | |||
22 | #include "sysinfo.h" | 22 | #include "sysinfo.h" |
23 | #include "systrace.h" | 23 | #include "systrace.h" |
24 | 24 | ||
25 | ProgressLoggerGui::ProgressLoggerGui(QWidget* parent): ProgressloggerInterface(parent) | 25 | ProgressLoggerGui::ProgressLoggerGui(QWidget* parent): QObject(parent) |
26 | { | 26 | { |
27 | downloadProgress = new QDialog(parent); | 27 | downloadProgress = new QDialog(parent); |
28 | downloadProgress->setModal(true); | 28 | downloadProgress->setModal(true); |
@@ -103,7 +103,7 @@ void ProgressLoggerGui::setRunning() | |||
103 | disconnect(dp.buttonAbort, &QAbstractButton::clicked, downloadProgress, &QWidget::close); | 103 | disconnect(dp.buttonAbort, &QAbstractButton::clicked, downloadProgress, &QWidget::close); |
104 | // emit aborted() once button is pressed but not closed(). | 104 | // emit aborted() once button is pressed but not closed(). |
105 | disconnect(dp.buttonAbort, &QAbstractButton::clicked, this, &ProgressLoggerGui::closed); | 105 | disconnect(dp.buttonAbort, &QAbstractButton::clicked, this, &ProgressLoggerGui::closed); |
106 | connect(dp.buttonAbort, SIGNAL(clicked()), this, SIGNAL(aborted())); | 106 | connect(dp.buttonAbort, &QAbstractButton::clicked, this, &ProgressLoggerGui::aborted); |
107 | 107 | ||
108 | } | 108 | } |
109 | 109 | ||
@@ -120,7 +120,7 @@ void ProgressLoggerGui::setFinished() | |||
120 | // close the window on button press. | 120 | // close the window on button press. |
121 | connect(dp.buttonAbort, &QAbstractButton::clicked, downloadProgress, &QWidget::close); | 121 | connect(dp.buttonAbort, &QAbstractButton::clicked, downloadProgress, &QWidget::close); |
122 | // emit closed() once button is pressed but not aborted(). | 122 | // emit closed() once button is pressed but not aborted(). |
123 | disconnect(dp.buttonAbort, SIGNAL(clicked()), this, SIGNAL(aborted())); | 123 | disconnect(dp.buttonAbort, &QAbstractButton::clicked, this, &ProgressLoggerGui::aborted); |
124 | connect(dp.buttonAbort, &QAbstractButton::clicked, this, &ProgressLoggerGui::closed); | 124 | connect(dp.buttonAbort, &QAbstractButton::clicked, this, &ProgressLoggerGui::closed); |
125 | } | 125 | } |
126 | 126 | ||
diff --git a/utils/rbutilqt/progressloggergui.h b/utils/rbutilqt/progressloggergui.h index ce9a8c9fa7..cc9b7df323 100644 --- a/utils/rbutilqt/progressloggergui.h +++ b/utils/rbutilqt/progressloggergui.h | |||
@@ -22,10 +22,10 @@ | |||
22 | 22 | ||
23 | #include <QWidget> | 23 | #include <QWidget> |
24 | 24 | ||
25 | #include "progressloggerinterface.h" | 25 | #include "progressloglevels.h" |
26 | #include "ui_progressloggerfrm.h" | 26 | #include "ui_progressloggerfrm.h" |
27 | 27 | ||
28 | class ProgressLoggerGui :public ProgressloggerInterface | 28 | class ProgressLoggerGui :public QObject |
29 | { | 29 | { |
30 | Q_OBJECT | 30 | Q_OBJECT |
31 | public: | 31 | public: |
diff --git a/utils/rbutilqt/rbutilqt.cpp b/utils/rbutilqt/rbutilqt.cpp index 1a0841c0f3..7c11f44806 100644 --- a/utils/rbutilqt/rbutilqt.cpp +++ b/utils/rbutilqt/rbutilqt.cpp | |||
@@ -42,8 +42,6 @@ | |||
42 | #include "backupdialog.h" | 42 | #include "backupdialog.h" |
43 | #include "changelog.h" | 43 | #include "changelog.h" |
44 | 44 | ||
45 | #include "progressloggerinterface.h" | ||
46 | |||
47 | #include "bootloaderinstallbase.h" | 45 | #include "bootloaderinstallbase.h" |
48 | #include "bootloaderinstallhelper.h" | 46 | #include "bootloaderinstallhelper.h" |
49 | 47 | ||