[PATCH 1/2] Include the ui_*.h headers in the main headers.

Thiago Macieira thiago at macieira.org
Thu Oct 3 11:54:24 UTC 2013


This means we can also remove the forward declarations.

This is the first step in removing the memory allocation for the ui
sub-classes. Without the second step, this commit is just making the
compilation time increase for no good reason :-)

Signed-off-by: Thiago Macieira <thiago at macieira.org>
---
 qt-ui/about.cpp                        | 1 -
 qt-ui/about.h                          | 5 +----
 qt-ui/divecomputermanagementdialog.cpp | 1 -
 qt-ui/divecomputermanagementdialog.h   | 4 +---
 qt-ui/diveplanner.cpp                  | 1 -
 qt-ui/diveplanner.h                    | 5 +----
 qt-ui/downloadfromdivecomputer.cpp     | 1 -
 qt-ui/downloadfromdivecomputer.h       | 4 +---
 qt-ui/maintab.cpp                      | 1 -
 qt-ui/maintab.h                        | 5 +----
 qt-ui/mainwindow.cpp                   | 1 -
 qt-ui/mainwindow.h                     | 7 ++-----
 qt-ui/preferences.cpp                  | 1 -
 qt-ui/preferences.h                    | 5 ++---
 qt-ui/printoptions.cpp                 | 1 -
 qt-ui/printoptions.h                   | 4 +---
 qt-ui/simplewidgets.cpp                | 2 --
 qt-ui/simplewidgets.h                  | 2 ++
 qt-ui/subsurfacewebservices.cpp        | 1 -
 qt-ui/tableview.cpp                    | 1 -
 qt-ui/tableview.h                      | 5 ++---
 21 files changed, 14 insertions(+), 44 deletions(-)

diff --git a/qt-ui/about.cpp b/qt-ui/about.cpp
index 821835f..56fae4b 100644
--- a/qt-ui/about.cpp
+++ b/qt-ui/about.cpp
@@ -1,5 +1,4 @@
 #include "about.h"
-#include "ui_about.h"
 #include "version.h"
 #include <QDebug>
 #include <QDialogButtonBox>
diff --git a/qt-ui/about.h b/qt-ui/about.h
index 6636b0e..2e6db60 100644
--- a/qt-ui/about.h
+++ b/qt-ui/about.h
@@ -3,10 +3,7 @@
 
 #include <QDialog>
 #include <QPushButton>
-
-namespace Ui {
-	class SubsurfaceAbout;
-}
+#include "ui_about.h"
 
 class SubsurfaceAbout : public QDialog {
 	Q_OBJECT
diff --git a/qt-ui/divecomputermanagementdialog.cpp b/qt-ui/divecomputermanagementdialog.cpp
index beb0ea4..9464810 100644
--- a/qt-ui/divecomputermanagementdialog.cpp
+++ b/qt-ui/divecomputermanagementdialog.cpp
@@ -1,6 +1,5 @@
 #include "divecomputermanagementdialog.h"
 #include "models.h"
-#include "ui_divecomputermanagementdialog.h"
 #include "mainwindow.h"
 #include <QMessageBox>
 #include "../qthelper.h"
diff --git a/qt-ui/divecomputermanagementdialog.h b/qt-ui/divecomputermanagementdialog.h
index 72d48cd..85b9cb7 100644
--- a/qt-ui/divecomputermanagementdialog.h
+++ b/qt-ui/divecomputermanagementdialog.h
@@ -1,12 +1,10 @@
 #ifndef DIVECOMPUTERMANAGEMENTDIALOG_H
 #define DIVECOMPUTERMANAGEMENTDIALOG_H
 #include <QDialog>
+#include "ui_divecomputermanagementdialog.h"
 
 class QModelIndex;
 class DiveComputerModel;
-namespace Ui{
-	class DiveComputerManagementDialog;
-};
 
 class DiveComputerManagementDialog : public QDialog{
 Q_OBJECT
diff --git a/qt-ui/diveplanner.cpp b/qt-ui/diveplanner.cpp
index 4fbdc5e..12b2d54 100644
--- a/qt-ui/diveplanner.cpp
+++ b/qt-ui/diveplanner.cpp
@@ -2,7 +2,6 @@
 #include "graphicsview-common.h"
 #include "models.h"
 #include "modeldelegates.h"
-#include "ui_diveplanner.h"
 #include "mainwindow.h"
 #include "maintab.h"
 #include "tableview.h"
diff --git a/qt-ui/diveplanner.h b/qt-ui/diveplanner.h
index e9fe205..7d9bb91 100644
--- a/qt-ui/diveplanner.h
+++ b/qt-ui/diveplanner.h
@@ -8,10 +8,7 @@
 #include <QDateTime>
 
 #include "dive.h"
-
-namespace Ui{
-	class DivePlanner;
-};
+#include "ui_diveplanner.h"
 
 class QListView;
 class QStringListModel;
diff --git a/qt-ui/downloadfromdivecomputer.cpp b/qt-ui/downloadfromdivecomputer.cpp
index eaf8efd..c10ec90 100644
--- a/qt-ui/downloadfromdivecomputer.cpp
+++ b/qt-ui/downloadfromdivecomputer.cpp
@@ -1,5 +1,4 @@
 #include "downloadfromdivecomputer.h"
-#include "ui_downloadfromdivecomputer.h"
 
 #include "../libdivecomputer.h"
 #include "../helpers.h"
diff --git a/qt-ui/downloadfromdivecomputer.h b/qt-ui/downloadfromdivecomputer.h
index bef3d54..2cd2975 100644
--- a/qt-ui/downloadfromdivecomputer.h
+++ b/qt-ui/downloadfromdivecomputer.h
@@ -6,10 +6,8 @@
 #include <QHash>
 #include <QMap>
 #include "../libdivecomputer.h"
+#include "ui_downloadfromdivecomputer.h"
 
-namespace Ui{
-	class DownloadFromDiveComputer;
-}
 struct device_data_t;
 
 class DownloadThread : public QThread{
diff --git a/qt-ui/maintab.cpp b/qt-ui/maintab.cpp
index 4512005..e9fc2ae 100644
--- a/qt-ui/maintab.cpp
+++ b/qt-ui/maintab.cpp
@@ -5,7 +5,6 @@
  *
  */
 #include "maintab.h"
-#include "ui_maintab.h"
 #include "mainwindow.h"
 #include "../helpers.h"
 #include "../statistics.h"
diff --git a/qt-ui/maintab.h b/qt-ui/maintab.h
index 5450c2d..f1a9815 100644
--- a/qt-ui/maintab.h
+++ b/qt-ui/maintab.h
@@ -12,13 +12,10 @@
 #include <QMap>
 
 #include "models.h"
+#include "ui_maintab.h"
 
 class QCompleter;
 struct dive;
-namespace Ui
-{
-	class MainTab;
-}
 
 struct NotesBackup{
 	QString airtemp;
diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp
index 8f2f823..e322569 100644
--- a/qt-ui/mainwindow.cpp
+++ b/qt-ui/mainwindow.cpp
@@ -4,7 +4,6 @@
  * classes for the main UI window in Subsurface
  */
 #include "mainwindow.h"
-#include "ui_mainwindow.h"
 
 #include <QVBoxLayout>
 #include <QFileDialog>
diff --git a/qt-ui/mainwindow.h b/qt-ui/mainwindow.h
index 7fc03b6..275416c 100644
--- a/qt-ui/mainwindow.h
+++ b/qt-ui/mainwindow.h
@@ -10,15 +10,12 @@
 #include <QMainWindow>
 #include <QAction>
 
+#include "ui_mainwindow.h"
+
 struct DiveList;
 class QSortFilterProxyModel;
 class DiveTripModel;
 
-namespace Ui
-{
-	class MainWindow;
-}
-
 class DiveInfo;
 class DiveNotes;
 class Stats;
diff --git a/qt-ui/preferences.cpp b/qt-ui/preferences.cpp
index f206224..0d49d49 100644
--- a/qt-ui/preferences.cpp
+++ b/qt-ui/preferences.cpp
@@ -1,5 +1,4 @@
 #include "preferences.h"
-#include "ui_preferences.h"
 #include <QSettings>
 #include <QDebug>
 
diff --git a/qt-ui/preferences.h b/qt-ui/preferences.h
index 171c41e..0d40b40 100644
--- a/qt-ui/preferences.h
+++ b/qt-ui/preferences.h
@@ -5,9 +5,8 @@
 #include "../dive.h"
 #include "../pref.h"
 
-namespace Ui{
-class PreferencesDialog;
-}
+#include "ui_preferences.h"
+
 class QAbstractButton;
 
 class PreferencesDialog :public QDialog{
diff --git a/qt-ui/printoptions.cpp b/qt-ui/printoptions.cpp
index cd169d3..437eb3f 100644
--- a/qt-ui/printoptions.cpp
+++ b/qt-ui/printoptions.cpp
@@ -1,5 +1,4 @@
 #include "printoptions.h"
-#include "ui_printoptions.h"
 #include "../display.h"
 
 PrintOptions::PrintOptions(QWidget *parent, struct options *printOpt)
diff --git a/qt-ui/printoptions.h b/qt-ui/printoptions.h
index cc0f4f6..86d9ba3 100644
--- a/qt-ui/printoptions.h
+++ b/qt-ui/printoptions.h
@@ -5,9 +5,7 @@
 #include <QSlider>
 #include <QLabel>
 
-namespace Ui {
-	class PrintOptions;
-};
+#include "ui_printoptions.h"
 
 // should be based on a custom QPrintDialog class
 class PrintOptions : public QWidget {
diff --git a/qt-ui/simplewidgets.cpp b/qt-ui/simplewidgets.cpp
index ce659c6..5b6c7a8 100644
--- a/qt-ui/simplewidgets.cpp
+++ b/qt-ui/simplewidgets.cpp
@@ -14,8 +14,6 @@
 
 #include "../dive.h"
 
-#include "ui_renumber.h"
-
 class MinMaxAvgWidgetPrivate{
 public:
 	QLabel *avgIco, *avgValue;
diff --git a/qt-ui/simplewidgets.h b/qt-ui/simplewidgets.h
index cf79d19..32b2998 100644
--- a/qt-ui/simplewidgets.h
+++ b/qt-ui/simplewidgets.h
@@ -7,6 +7,8 @@ class QAbstractButton;
 #include <QWidget>
 #include <QDialog>
 
+#include "ui_renumber.h"
+
 class MinMaxAvgWidget : public QWidget{
 	Q_OBJECT
 	Q_PROPERTY(double minimum READ minimum WRITE setMinimum)
diff --git a/qt-ui/subsurfacewebservices.cpp b/qt-ui/subsurfacewebservices.cpp
index cdf8ee6..cdc5f5a 100644
--- a/qt-ui/subsurfacewebservices.cpp
+++ b/qt-ui/subsurfacewebservices.cpp
@@ -1,5 +1,4 @@
 #include "subsurfacewebservices.h"
-#include "ui_subsurfacewebservices.h"
 #include "../webservice.h"
 
 #include <libxml/parser.h>
diff --git a/qt-ui/tableview.cpp b/qt-ui/tableview.cpp
index 02bf208..b3d2c36 100644
--- a/qt-ui/tableview.cpp
+++ b/qt-ui/tableview.cpp
@@ -1,5 +1,4 @@
 #include "tableview.h"
-#include "ui_tableview.h"
 #include "models.h"
 
 #include <QPushButton>
diff --git a/qt-ui/tableview.h b/qt-ui/tableview.h
index d22f466..0baac08 100644
--- a/qt-ui/tableview.h
+++ b/qt-ui/tableview.h
@@ -7,13 +7,12 @@
  */
 #include <QWidget>
 
+#include "ui_tableview.h"
+
 class QPushButton;
 class QAbstractItemModel;
 class QModelIndex;
 class QTableView;
-namespace Ui{
-	class TableView;
-};
 
 class TableView : public QWidget {
 Q_OBJECT
-- 
1.7.11.7



More information about the subsurface mailing list