Bootstrap: remove QDataStream

It was only used by the cmake_automoc_parser so it would write a 64-bit
in big-endian format. So bypass QDataStream and write it native
endianness.

Change-Id: I01ec3c774d9943adb903fffd17b79c78e56db4cf
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by:  Alexey Edelev <alexey.edelev@qt.io>

src/corelib/kernel/qmetatype.cpp

@@ -22,13 +22,14 @@
 #endif
 #include "quuid.h"
 #include "qvariant.h"
-#include "qdatastream.h"
 
 #if QT_CONFIG(regularexpression)
 #  include "qregularexpression.h"
 #endif
 
 #ifndef QT_BOOTSTRAPPED
+#  include "qdatastream.h"
+
 #  include "qbitarray.h"
 #  include "qurl.h"
 #  include "qvariant.h"

src/corelib/kernel/qmetatype.h

@@ -2368,19 +2368,21 @@ struct QDebugStreamOperatorForType <T, false>
 
 template<typename T, bool = QTypeTraits::has_stream_operator_v<QDataStream, T>>
 struct QDataStreamOperatorForType
+{
+    static constexpr QMetaTypeInterface::DataStreamOutFn dataStreamOut = nullptr;
+    static constexpr QMetaTypeInterface::DataStreamInFn dataStreamIn = nullptr;
+};
+
+#ifndef QT_NO_DATASTREAM
+template<typename T>
+struct QDataStreamOperatorForType <T, true>
 {
     static void dataStreamOut(const QMetaTypeInterface *, QDataStream &ds, const void *a)
     { ds << *reinterpret_cast<const T *>(a); }
     static void dataStreamIn(const QMetaTypeInterface *, QDataStream &ds, void *a)
     { ds >> *reinterpret_cast<T *>(a); }
 };
-
-template<typename T>
-struct QDataStreamOperatorForType <T, false>
-{
-    static constexpr QMetaTypeInterface::DataStreamOutFn dataStreamOut = nullptr;
-    static constexpr QMetaTypeInterface::DataStreamInFn dataStreamIn = nullptr;
-};
+#endif
 
 // Performance optimization:
 //

src/corelib/serialization/qdatastream.h

@@ -25,7 +25,7 @@ class QDataStream;
 class QIODevice;
 class QString;
 
-#if !defined(QT_NO_DATASTREAM) || defined(QT_BOOTSTRAPPED)
+#if !defined(QT_NO_DATASTREAM)
 class QDataStreamPrivate;
 namespace QtPrivate {
 class StreamStateSaver;

src/corelib/text/qbytearray.cpp

@@ -3329,7 +3329,7 @@ void QByteArray::clear()
     d.clear();
 }
 
-#if !defined(QT_NO_DATASTREAM) || defined(QT_BOOTSTRAPPED)
+#if !defined(QT_NO_DATASTREAM)
 
 /*! \relates QByteArray
 

src/corelib/text/qstring.cpp

@@ -9549,7 +9549,7 @@ QString &QString::setRawData(const QChar *unicode, qsizetype size)
     \sa toUcs4(), toStdWString(), toStdU16String()
 */
 
-#if !defined(QT_NO_DATASTREAM) || defined(QT_BOOTSTRAPPED)
+#if !defined(QT_NO_DATASTREAM)
 /*!
     \fn QDataStream &operator<<(QDataStream &stream, const QString &string)
     \relates QString

src/tools/bootstrap/CMakeLists.txt

@@ -51,7 +51,6 @@ qt_internal_extend_target(Bootstrap
         ../../corelib/serialization/qcborcommon.cpp
         ../../corelib/serialization/qcborstreamwriter.cpp
         ../../corelib/serialization/qcborvalue.cpp
-        ../../corelib/serialization/qdatastream.cpp
         ../../corelib/serialization/qjsonarray.cpp
         ../../corelib/serialization/qjsoncbor.cpp
         ../../corelib/serialization/qjsondocument.cpp

src/tools/cmake_automoc_parser/main.cpp

@@ -185,11 +185,8 @@ static bool writeJsonFiles(const QList<QString> &fileList, const QString &fileLi
     }
 
     qint64 timestamp = std::numeric_limits<qint64>::min();
-    QByteArray timestampBuffer = timestampFile.readAll();
-    if (timestampBuffer.size() == sizeof(timestamp)) {
-        QDataStream istream(&timestampBuffer, QIODevice::ReadOnly);
-        istream >> timestamp;
-    }
+    if (timestampFile.size() == sizeof(timestamp))
+        timestampFile.read(reinterpret_cast<char *>(&timestamp), sizeof(timestamp));
 
     // Check if any of the metatype json files produced by automoc is newer than the last file
     // processed by cmake_automoc parser
@@ -215,11 +212,8 @@ static bool writeJsonFiles(const QList<QString> &fileList, const QString &fileLi
         textStream.flush();
 
         // Update the timestamp according the newest json file timestamp.
-        timestampBuffer.clear();
-        QDataStream ostream(&timestampBuffer, QIODevice::WriteOnly);
-        ostream << timestamp;
         timestampFile.resize(0);
-        timestampFile.write(timestampBuffer);
+        timestampFile.write(reinterpret_cast<char *>(&timestamp), sizeof(timestamp));
     }
     return true;
 }