Utils: More forward declarations / drop unused headers [L-O]

Round 1 - focus on headers.
For classes with initial in range [L-O].
Try to keep the same separators between different kind of headers.

Drop changes to NameValueDictionary for now, as apparently
environmentfwd.h is broken currently. It looks we can't
forward declare the argument T inside QList<T> - the type
must be complete.

Change-Id: If26e88357a2ffbb91a79c4d003046443d98d5673
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: hjk <hjk@qt.io>

1 files changed, 8 insertions, 4 deletions

diff --git a/src/libs/utils/namevaluevalidator.h b/src/libs/utils/namevaluevalidator.h
index c81dbe2e5b..dc4952569d 100644
--- a/src/libs/utils/namevaluevalidator.h
+++ b/src/libs/utils/namevaluevalidator.h
@@ -25,22 +25,26 @@
 
 #pragma once
 
-#include "environmentfwd.h"
 #include "utils_global.h"
 
-#include <QModelIndex>
+#include "environmentfwd.h"
+
 #include <QPersistentModelIndex>
 #include <QTimer>
 #include <QValidator>
 
+QT_FORWARD_DECLARE_CLASS(QModelIndex)
+
 namespace Utils {
 
+class NameValueModel;
+
 class QTCREATOR_UTILS_EXPORT NameValueValidator : public QValidator
 {
     Q_OBJECT
 public:
     NameValueValidator(QWidget *parent,
-                       Utils::NameValueModel *model,
+                       NameValueModel *model,
                        QTreeView *view,
                        const QModelIndex &index,
                        const QString &toolTipText);
@@ -51,7 +55,7 @@ public:
 
 private:
     const QString m_toolTipText;
-    Utils::NameValueModel *m_model;
+    NameValueModel *m_model;
     QTreeView *m_view;
     QPersistentModelIndex m_index;
     mutable QTimer m_hideTipTimer;