DiffEditor: Remove a bit of fluff

Change-Id: Ie1b2df8eb336078ced59d1406020ae57d2b53682
Reviewed-by: Jarek Kobus <jaroslaw.kobus@theqtcompany.com>

src/plugins/diffeditor/diffeditor.cpp

@@ -263,7 +263,6 @@ DiffEditor::DiffEditor(const QSharedPointer<DiffEditorDocument> &doc)
 DiffEditor::~DiffEditor()
 {
     delete m_toolBar;
-    if (m_widget)
     delete m_widget;
 }
 

src/plugins/diffeditor/diffeditordocument.cpp

@@ -53,10 +53,6 @@ DiffEditorDocument::DiffEditorDocument() :
     setTemporary(true);
 }
 
-DiffEditorDocument::~DiffEditorDocument()
-{
-}
-
 DiffEditorController *DiffEditorDocument::controller() const
 {
     return m_controller;

src/plugins/diffeditor/diffeditordocument.h

@@ -44,8 +44,7 @@ class DIFFEDITOR_EXPORT DiffEditorDocument : public Core::BaseTextDocument
     Q_OBJECT
     Q_PROPERTY(QString plainText READ plainText STORED false) // For access by code pasters
 public:
-    explicit DiffEditorDocument();
-    virtual ~DiffEditorDocument();
+    DiffEditorDocument();
 
     DiffEditorController *controller() const;
 
@@ -62,7 +61,7 @@ public:
     QString plainText() const;
 
 private:
-    DiffEditorController *m_controller;
+    DiffEditorController *const m_controller;
 };
 
 } // namespace DiffEditor

src/plugins/diffeditor/diffeditorguicontroller.cpp

@@ -33,6 +33,8 @@
 
 #include <coreplugin/icore.h>
 
+#include <utils/qtcassert.h>
+
 static const char settingsGroupC[] = "DiffEditor";
 static const char descriptionVisibleKeyC[] = "DescriptionVisible";
 static const char horizontalScrollBarSynchronizationKeyC[] =
@@ -49,6 +51,7 @@ DiffEditorGuiController::DiffEditorGuiController(
       m_syncScrollBars(true),
       m_currentDiffFileIndex(-1)
 {
+    QTC_ASSERT(m_controller, return);
     QSettings *s = Core::ICore::settings();
     s->beginGroup(QLatin1String(settingsGroupC));
     m_descriptionVisible = s->value(QLatin1String(descriptionVisibleKeyC),

src/plugins/diffeditor/diffeditorguicontroller.h

@@ -66,7 +66,7 @@ private slots:
     void slotUpdateDiffFileIndex();
 
 private:
-    DiffEditorController *m_controller;
+    DiffEditorController *const m_controller;
     bool m_descriptionVisible;
     bool m_syncScrollBars;
     int m_currentDiffFileIndex;

src/plugins/diffeditor/sidebysidediffeditorwidget.cpp

@@ -826,11 +826,6 @@ SideBySideDiffEditorWidget::SideBySideDiffEditorWidget(QWidget *parent)
     clear(tr("No controller"));
 }
 
-SideBySideDiffEditorWidget::~SideBySideDiffEditorWidget()
-{
-
-}
-
 void SideBySideDiffEditorWidget::setDiffEditorGuiController(
         DiffEditorGuiController *controller)
 {
@@ -882,7 +877,6 @@ void SideBySideDiffEditorWidget::setDiffEditorGuiController(
     }
 }
 
-
 DiffEditorGuiController *SideBySideDiffEditorWidget::diffEditorGuiController() const
 {
     return m_guiController;

src/plugins/diffeditor/sidebysidediffeditorwidget.h

@@ -55,8 +55,7 @@ class DIFFEDITOR_EXPORT SideBySideDiffEditorWidget : public QWidget
 {
     Q_OBJECT
 public:
-    SideBySideDiffEditorWidget(QWidget *parent = 0);
-    ~SideBySideDiffEditorWidget();
+    explicit SideBySideDiffEditorWidget(QWidget *parent = 0);
 
     void setDiffEditorGuiController(DiffEditorGuiController *controller);
     DiffEditorGuiController *diffEditorGuiController() const;