Change static member of SharedStrings to normal member

src/xlsx/xlsxsharedstrings.cpp

@@ -146,10 +146,8 @@ QByteArray SharedStrings::saveToXmlData() const
     return data;
 }
 
-QSharedPointer<SharedStrings> SharedStrings::loadFromXmlFile(QIODevice *device)
+bool SharedStrings::loadFromXmlFile(QIODevice *device)
 {
-    QSharedPointer<SharedStrings> sst(new SharedStrings);
-
     XmlStreamReader reader(device);
     int count = 0;
     while(!reader.atEnd()) {
@@ -163,23 +161,24 @@ QSharedPointer<SharedStrings> SharedStrings::loadFromXmlFile(QIODevice *device)
                      if (reader.name() == QLatin1String("t")) {
 //                         QXmlStreamAttributes attributes = reader.attributes();
                          QString string = reader.readElementText();
-                         int idx = sst->m_stringList.size();
-                         sst->m_stringTable[string] = XlsxSharedStringInfo(idx, 0);
-                         sst->m_stringList.append(string);
+                         int idx = m_stringList.size();
+                         m_stringTable[string] = XlsxSharedStringInfo(idx, 0);
+                         m_stringList.append(string);
                      }
                  }
              }
          }
     }
 
-    if (sst->m_stringTable.size() != count) {
+    if (m_stringTable.size() != count) {
         qDebug("Error: Shared string count");
+        return false;
     }
 
-    return sst;
+    return true;
 }
 
-QSharedPointer<SharedStrings> SharedStrings::loadFromXmlData(const QByteArray &data)
+bool SharedStrings::loadFromXmlData(const QByteArray &data)
 {
     QBuffer buffer;
     buffer.setData(data);

src/xlsx/xlsxsharedstrings_p.h

@@ -62,8 +62,8 @@ public:
 
     void saveToXmlFile(QIODevice *device) const;
     QByteArray saveToXmlData() const;
-    static QSharedPointer<SharedStrings> loadFromXmlFile(QIODevice *device);
-    static QSharedPointer<SharedStrings> loadFromXmlData(const QByteArray &data);
+    bool loadFromXmlFile(QIODevice *device);
+    bool loadFromXmlData(const QByteArray &data);
 
 private:
     QHash<QString, XlsxSharedStringInfo> m_stringTable; //for fast lookup

tests/auto/sharedstrings/tst_sharedstringstest.cpp

@@ -92,7 +92,8 @@ void SharedStringsTest::testLoadXmlData()
     sst.addSharedString("Hello Qt!");
     QByteArray xmlData = sst.saveToXmlData();
 
-    QSharedPointer<QXlsx::SharedStrings> sst2 = QXlsx::SharedStrings::loadFromXmlData(xmlData);
+    QSharedPointer<QXlsx::SharedStrings> sst2(new QXlsx::SharedStrings);
+    sst2->loadFromXmlData(xmlData);
 
     QCOMPARE(sst2->getSharedString(0), QStringLiteral("Hello Qt!"));
     QCOMPARE(sst2->getSharedString(2), QStringLiteral("Hello World"));