Skip to content

Commit 3261f84

Browse files
committed
GUI: Fix includes for dialogs/views.
At first we created the GUI in the code and so needed all the layouts etc include to the code. Now when we are using UI files we don't need all those included. So clean them up. Also use forward declarations instead of includes in header files when possible.
1 parent bc8242d commit 3261f84

10 files changed

Lines changed: 21 additions & 27 deletions

gui/aboutdialog.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,9 @@
1515
* You should have received a copy of the GNU General Public License
1616
* along with this program. If not, see <http://www.gnu.org/licenses/>.
1717
*/
18+
1819
#include <QDialog>
19-
#include <QVBoxLayout>
20-
#include <QHBoxLayout>
21-
#include <QLabel>
22-
#include <QPushButton>
20+
#include <QWidget>
2321
#include "aboutdialog.h"
2422

2523
AboutDialog::AboutDialog(const QString &version, QWidget *parent)

gui/aboutdialog.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,10 @@
2121

2222
#include <QDialog>
2323
#include <QString>
24-
2524
#include "ui_about.h"
2625

26+
class QWidget;
27+
2728
/// @addtogroup GUI
2829
/// @{
2930

gui/applicationdialog.cpp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,11 @@
1616
* along with this program. If not, see <http://www.gnu.org/licenses/>.
1717
*/
1818

19-
#include "applicationdialog.h"
20-
#include <QVBoxLayout>
21-
#include <QPushButton>
22-
#include <QHBoxLayout>
23-
#include <QLabel>
19+
#include <QWidget>
2420
#include <QFileDialog>
2521
#include <QDebug>
2622
#include <QMessageBox>
23+
#include "applicationdialog.h"
2724

2825

2926
ApplicationDialog::ApplicationDialog(const QString &name,

gui/applicationdialog.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,11 @@
2121

2222
#include <QDialog>
2323
#include <QLineEdit>
24+
#include <QString>
2425
#include "ui_application.h"
2526

27+
class QWidget;
28+
2629
/// @addtogroup GUI
2730
/// @{
2831

gui/fileviewdialog.cpp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,10 @@
1616
* along with this program. If not, see <http://www.gnu.org/licenses/>.
1717
*/
1818

19-
#include <QDialog>
20-
#include <QVBoxLayout>
21-
#include <QHBoxLayout>
22-
#include <QTextEdit>
23-
#include <QPushButton>
2419
#include <QFile>
2520
#include <QByteArray>
2621
#include <QMessageBox>
22+
#include <QTextEdit>
2723
#include "fileviewdialog.h"
2824

2925
FileViewDialog::FileViewDialog(const QString &file,

gui/fileviewdialog.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@
2323
#include <QString>
2424
#include "ui_file.h"
2525

26+
class QWidget;
27+
class QTextEdit;
28+
2629
/// @addtogroup GUI
2730
/// @{
2831

gui/resultsview.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,11 @@
1717
*/
1818

1919
#include <QDebug>
20-
#include <QVBoxLayout>
2120
#include <QFile>
2221
#include <QMessageBox>
2322
#include "erroritem.h"
2423
#include "resultsview.h"
24+
#include "resultstree.h"
2525
#include "report.h"
2626
#include "txtreport.h"
2727
#include "xmlreport.h"

gui/resultsview.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
#include <QWidget>
2525
#include <QProgressBar>
2626
#include "../lib/errorlogger.h"
27-
#include "resultstree.h"
2827
#include "common.h"
2928
#include "report.h"
3029
#include "ui_resultsview.h"

gui/settingsdialog.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,15 @@
1717
*/
1818

1919

20-
#include "settingsdialog.h"
2120
#include <QDialog>
2221
#include <QWidget>
2322
#include <QLabel>
2423
#include <QDebug>
2524
#include <QTabWidget>
25+
#include <QSettings>
26+
#include "settingsdialog.h"
2627
#include "applicationdialog.h"
28+
#include "applicationlist.h"
2729
#include "common.h"
2830

2931
SettingsDialog::SettingsDialog(QSettings *programSettings,

gui/settingsdialog.h

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,18 +21,13 @@
2121
#define SETTINGSDIALOG_H
2222

2323
#include <QDialog>
24-
#include <QLineEdit>
25-
#include <QComboBox>
26-
#include <QSettings>
27-
#include <QCheckBox>
28-
#include <QVBoxLayout>
29-
#include <QPushButton>
30-
#include "applicationlist.h"
31-
32-
#include <QListWidget>
3324
#include <QKeyEvent>
3425
#include "ui_settings.h"
3526

27+
class QSettings;
28+
class QWidget;
29+
class ApplicationList;
30+
3631
/// @addtogroup GUI
3732
/// @{
3833

0 commit comments

Comments
 (0)