From 5c2ff22ba117f295718c529198ab42ee4646d90c Mon Sep 17 00:00:00 2001 From: Thiago Macieira Date: Tue, 26 Apr 2016 16:40:37 -0700 Subject: Use void instead of uchar in the endian-swapping function parameters This allows us to pass pointers to storage that is not an array of uchar, which it hardly ever is. Change-Id: Ifea6e497f11a461db432ffff14490d2c2df21906 Reviewed-by: Konstantin Ritt Reviewed-by: Marc Mutz --- src/network/access/qspdyprotocolhandler.cpp | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'src/network/access') diff --git a/src/network/access/qspdyprotocolhandler.cpp b/src/network/access/qspdyprotocolhandler.cpp index e4e058b772..413b03cc22 100644 --- a/src/network/access/qspdyprotocolhandler.cpp +++ b/src/network/access/qspdyprotocolhandler.cpp @@ -364,43 +364,43 @@ void QSpdyProtocolHandler::_q_readyRead() static qint16 twoBytesToInt(const char *bytes) { - return qFromBigEndian(reinterpret_cast(bytes)); + return qFromBigEndian(bytes); } static qint32 threeBytesToInt(const char *bytes) { - return qFromBigEndian(reinterpret_cast(bytes)) >> 8; + return qFromBigEndian(bytes) >> 8; } static qint32 fourBytesToInt(const char *bytes) { - return qFromBigEndian(reinterpret_cast(bytes)); + return qFromBigEndian(bytes); } static void appendIntToThreeBytes(char *output, qint32 number) { - qToBigEndian(number, reinterpret_cast(output + 1)); - qToBigEndian(number >> 16, reinterpret_cast(output)); + qToBigEndian(number, output + 1); + qToBigEndian(number >> 16, output); } static void appendIntToFourBytes(char *output, qint32 number) { - qToBigEndian(number, reinterpret_cast(output)); + qToBigEndian(number, output); } static QByteArray intToFourBytes(qint32 number) // ### try to use appendIntToFourBytes where possible { - uchar data[4]; + char data[4]; qToBigEndian(number, data); - QByteArray ret(reinterpret_cast(data), 4); + QByteArray ret(data, 4); return ret; } static QByteArray intToThreeBytes(qint32 number) { - uchar data[4]; + char data[4]; qToBigEndian(number << 8, data); - QByteArray ret(reinterpret_cast(data), 3); + QByteArray ret(data, 3); return ret; } -- cgit v1.2.3 From 874eb8b44b3ca9cfab63372af10886d16dfc327d Mon Sep 17 00:00:00 2001 From: Anton Kudryavtsev Date: Thu, 12 May 2016 19:02:33 +0300 Subject: QtNetwork: use QStringRef to optimize memory allocation Replace substring functions that return QString with corresponding functions that return QStringRef where it's possible. Change-Id: Id7f70e5bd72c91ed210ec463aad1d84fe3b40c15 Reviewed-by: Timur Pocheptsov --- src/network/access/qftp.cpp | 2 +- src/network/access/qnetworkcookiejar.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'src/network/access') diff --git a/src/network/access/qftp.cpp b/src/network/access/qftp.cpp index 23ec390ae5..19b519243c 100644 --- a/src/network/access/qftp.cpp +++ b/src/network/access/qftp.cpp @@ -1218,7 +1218,7 @@ bool QFtpPI::startNextCmd() pendingCommands.pop_front(); #if defined(QFTPPI_DEBUG) - qDebug("QFtpPI send: %s", currentCmd.left(currentCmd.length()-2).toLatin1().constData()); + qDebug("QFtpPI send: %s", currentCmd.leftRef(currentCmd.length() - 2).toLatin1().constData()); #endif state = Waiting; commandSocket.write(currentCmd.toUtf8()); diff --git a/src/network/access/qnetworkcookiejar.cpp b/src/network/access/qnetworkcookiejar.cpp index 283dd3509e..429b71eb21 100644 --- a/src/network/access/qnetworkcookiejar.cpp +++ b/src/network/access/qnetworkcookiejar.cpp @@ -160,7 +160,7 @@ static inline bool isParentDomain(const QString &domain, const QString &referenc if (!reference.startsWith(QLatin1Char('.'))) return domain == reference; - return domain.endsWith(reference) || domain == reference.mid(1); + return domain.endsWith(reference) || domain == reference.midRef(1); } /*! -- cgit v1.2.3