Merge two std::map, they had mostly the same contents

- Moved to QtUtils, so it can be shared with more checks
- Removed the duplicate one from foreach, which was even wrong since
it included QVarLengthArray, but that container isn't implicitly shared

kfunk: ^^

1 files changed, 26 insertions, 2 deletions

diff --git a/src/QtUtils.cpp b/src/QtUtils.cpp
index 5d1ffbb2..6bda1eb0 100644
--- a/src/QtUtils.cpp
+++ b/src/QtUtils.cpp
@@ -58,6 +58,32 @@ const vector<string> & QtUtils::qtCOWContainers()
     return classes;
 }
 
+
+std::map<string, std::vector<string> > QtUtils::detachingMethods()
+{
+    static std::map<string, std::vector<string> > map;
+    if (map.empty()) {
+        map["QList"] = {"first", "last", "begin", "end", "front", "back", "operator[]"};
+        map["QVector"] = {"first", "last", "begin", "end", "front", "back", "data", "operator[]", "fill" };
+        map["QMap"] = {"begin", "end", "first", "find", "last", "lowerBound", "upperBound", "operator[]" };
+        map["QHash"] = {"begin", "end", "find", "operator[]" };
+        map["QLinkedList"] = {"first", "last", "begin", "end", "front", "back", "operator[]" };
+        map["QSet"] = {"begin", "end", "find", "operator[]" };
+        map["QStack"] = map["QVector"];
+        map["QStack"].push_back({"top"});
+        map["QQueue"] = map["QVector"];
+        map["QQueue"].push_back({"head"});
+        map["QMultiMap"] = map["QMap"];
+        map["QMultiHash"] = map["QHash"];
+        map["QString"] = {"begin", "end", "data", "operator[]"};
+        map["QByteArray"] = {"data", "operator[]"};
+        map["QImage"] = {"bits", "scanLine"};
+    }
+
+    return map;
+}
+
+
 bool QtUtils::isQtCOWIterableClass(clang::CXXRecordDecl *record)
 {
     if (!record)
@@ -297,5 +323,3 @@ CXXMethodDecl *QtUtils::pmfFromUnary(UnaryOperator *uo)
 
     return nullptr;
 }
-
-