Fix: remove baseline checksums before adding rendered

.. otherwise, the comparison will always yield true...

Also a simplification: QList provides default datastreaming ops

2 files changed, 0 insertions, 25 deletions

diff --git a/tests/arthur/common/baselineprotocol.cpp b/tests/arthur/common/baselineprotocol.cpp
index cbead24..7996fff 100644
--- a/tests/arthur/common/baselineprotocol.cpp
+++ b/tests/arthur/common/baselineprotocol.cpp
@@ -129,26 +129,6 @@ QDataStream & operator>> (QDataStream &stream, ImageItem &ii)
     return stream;
 }
 
-QDataStream & operator<< (QDataStream &stream, const QList<quint64> &checkSumList)
-{
-    stream << quint32(checkSumList.count());
-    foreach(quint64 checksum, checkSumList)
-        stream << checksum;
-    return stream;
-}
-
-QDataStream & operator>> (QDataStream &stream, QList<quint64> &checkSumList)
-{
-    quint32 numChecksums;
-    stream >> numChecksums;
-    quint64 checkSum;
-    for (int i=0; i<(int)numChecksums; ++i) {
-        stream >> checkSum;
-        checkSumList.append(checkSum);
-    }
-    return stream;
-}
-
 BaselineProtocol::~BaselineProtocol()
 {
     socket.close();
diff --git a/tests/arthur/common/baselineprotocol.h b/tests/arthur/common/baselineprotocol.h
index 1cd8bb6..5427023 100644
--- a/tests/arthur/common/baselineprotocol.h
+++ b/tests/arthur/common/baselineprotocol.h
@@ -57,15 +57,10 @@ public:
 QDataStream & operator<< (QDataStream &stream, const ImageItem &ii);
 QDataStream & operator>> (QDataStream& stream, ImageItem& ii);
 
-
 Q_DECLARE_METATYPE(ImageItem);
 
-QDataStream & operator<< (QDataStream &stream, const QList<quint64> &checkSumList);
-QDataStream & operator>> (QDataStream &stream, QList<quint64> &checkSumList);
-
 typedef QVector<ImageItem> ImageItemList;
 
-
 class BaselineProtocol
 {
 public: