TextEditor: De-qobjectify SnippetProvider

It's really not much more than a struct with three items.
And there's no need to have them in the global object pool.

Change-Id: Ie4f46ba2ecb1b90ef22479a0e81d03d9301c2e7a
Reviewed-by: David Schulz <david.schulz@qt.io>

1 files changed, 5 insertions, 7 deletions

diff --git a/src/plugins/texteditor/snippets/snippetprovider.h b/src/plugins/texteditor/snippets/snippetprovider.h
index 5c343df182..d33a3bcdd0 100644
--- a/src/plugins/texteditor/snippets/snippetprovider.h
+++ b/src/plugins/texteditor/snippets/snippetprovider.h
@@ -27,7 +27,7 @@
 
 #include <texteditor/texteditor_global.h>
 
-#include <QObject>
+#include <QString>
 
 #include <functional>
 
@@ -35,13 +35,14 @@ namespace TextEditor {
 
 class TextEditorWidget;
 
-class TEXTEDITOR_EXPORT SnippetProvider : public QObject
+class TEXTEDITOR_EXPORT SnippetProvider
 {
-    Q_OBJECT
 public:
+    SnippetProvider() = default;
+
     using EditorDecorator = std::function<void(TextEditorWidget *)>;
 
-    static const QList<SnippetProvider *> snippetProviders();
+    static const QList<SnippetProvider> &snippetProviders();
     static SnippetProvider *snippetProviderForGroupId(const QString &groupId);
 
     static void registerGroup(const QString &groupId, const QString &displayName,
@@ -54,9 +55,6 @@ public:
     void decorateEditor(TextEditorWidget *editor) const;
 
 private:
-    SnippetProvider();
-    ~SnippetProvider() override;
-
     QString m_groupId;
     QString m_displayName;
     EditorDecorator m_editorDecorator;