CppEditor: simplify CppQuickFixOperation interface

Change-Id: Ib3ed82c7f07f80027b18471ffb7b3055fa74eb52
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

1 files changed, 7 insertions, 37 deletions

diff --git a/src/plugins/cppeditor/cppquickfix.cpp b/src/plugins/cppeditor/cppquickfix.cpp
index b510eb7117..2720cf86a3 100644
--- a/src/plugins/cppeditor/cppquickfix.cpp
+++ b/src/plugins/cppeditor/cppquickfix.cpp
@@ -53,23 +53,15 @@ using namespace CppEditor::Internal;
 using namespace CppTools;
 using namespace TextEditor;
 using namespace CPlusPlus;
-using namespace Utils;
 
-CppQuickFixOperation::CppQuickFixOperation(
-        const QSharedPointer<const CppQuickFixAssistInterface> &interface, int priority)
+CppQuickFixOperation::CppQuickFixOperation(const CppQuickFixInterface &interface, int priority)
     : QuickFixOperation(priority)
     , m_interface(interface)
 {}
 
-CppQuickFixOperation::~CppQuickFixOperation()
-{}
-
-void CppQuickFixOperation::perform()
+Snapshot CppQuickFixOperation::snapshot() const
 {
-    CppRefactoringChanges refactoring(m_interface->snapshot());
-    CppRefactoringFilePtr current = refactoring.file(fileName());
-
-    performChanges(current, refactoring);
+    return m_interface->snapshot();
 }
 
 const CppQuickFixAssistInterface *CppQuickFixOperation::assistInterface() const
@@ -82,32 +74,10 @@ QString CppQuickFixOperation::fileName() const
     return m_interface->document()->fileName();
 }
 
-CppQuickFixFactory::CppQuickFixFactory()
-{
-}
-
-CppQuickFixFactory::~CppQuickFixFactory()
+void CppQuickFixFactory::matchingOperations(const QuickFixInterface &interface, QuickFixOperations &result)
 {
-}
-
-QList<QuickFixOperation::Ptr> CppQuickFixFactory::matchingOperations(
-    const QSharedPointer<const TextEditor::IAssistInterface> &interface)
-{
-    QSharedPointer<const CppQuickFixAssistInterface> cppInterface =
-            interface.staticCast<const CppQuickFixAssistInterface>();
+    CppQuickFixInterface cppInterface = interface.staticCast<const CppQuickFixAssistInterface>();
     if (cppInterface->path().isEmpty())
-        return QList<QuickFixOperation::Ptr>();
-    return match(cppInterface);
-}
-
-QList<CppQuickFixOperation::Ptr> CppQuickFixFactory::singleResult(CppQuickFixOperation *operation)
-{
-    QList<CppQuickFixOperation::Ptr> result;
-    result.append(CppQuickFixOperation::Ptr(operation));
-    return result;
-}
-
-QList<CppQuickFixOperation::Ptr> CppQuickFixFactory::noResult()
-{
-    return QList<CppQuickFixOperation::Ptr>();
+        return;
+    match(cppInterface, result);
 }