Whitespace cleanup in corelib/iov6.0.0-beta2

Selective application of clang-format to follow our coding style where it is significantly off. Change-Id: I0ff4ed146fe53922691d5473d0c236f31d478a04 Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io> Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io> 

18 files changed, 224 insertions, 224 deletions

diff --git a/src/corelib/io/qbuffer.h b/src/corelib/io/qbuffer.h
index 39e1e7b39d8..c2aba92a10f 100644
--- a/src/corelib/io/qbuffer.h
+++ b/src/corelib/io/qbuffer.h
@@ -45,7 +45,6 @@

 QT_BEGIN_NAMESPACE

-
 class QObject;
 class QBufferPrivate;

@@ -57,11 +56,11 @@ class Q_CORE_EXPORT QBuffer : public QIODevice

 public:
 #ifndef QT_NO_QOBJECT
- explicit QBuffer(QObject *parent = nullptr);
- QBuffer(QByteArray *buf, QObject *parent = nullptr);
+ explicit QBuffer(QObject *parent = nullptr);
+ QBuffer(QByteArray *buf, QObject *parent = nullptr);
 #else
- QBuffer();
- explicit QBuffer(QByteArray *buf);
+ QBuffer();
+ explicit QBuffer(QByteArray *buf);
 #endif
 ~QBuffer();

diff --git a/src/corelib/io/qdir.h b/src/corelib/io/qdir.h
index 92b3faee7eb..86ab33a2fcf 100644
--- a/src/corelib/io/qdir.h
+++ b/src/corelib/io/qdir.h
@@ -213,7 +213,7 @@ public:
 bool makeAbsolute();

 bool operator==(const QDir &dir) const;
- inline bool operator!=(const QDir &dir) const { return !operator==(dir); }
+ inline bool operator!=(const QDir &dir) const { return !operator==(dir); }

 bool remove(const QString &fileName);
 bool rename(const QString &oldName, const QString &newName);
@@ -266,12 +266,8 @@ protected:
 private:
 friend class QDirIterator;
 // Q_DECLARE_PRIVATE equivalent for shared data pointers
- QDirPrivate* d_func();
- inline const QDirPrivate* d_func() const
- {
- return d_ptr.constData();
- }
-
+ QDirPrivate *d_func();
+ const QDirPrivate *d_func() const { return d_ptr.constData(); }
 };

 Q_DECLARE_SHARED(QDir)
diff --git a/src/corelib/io/qdiriterator.h b/src/corelib/io/qdiriterator.h
index a8f769eac08..acfe040a262 100644
--- a/src/corelib/io/qdiriterator.h
+++ b/src/corelib/io/qdiriterator.h
@@ -44,9 +44,9 @@

 QT_BEGIN_NAMESPACE

-
 class QDirIteratorPrivate;
-class Q_CORE_EXPORT QDirIterator {
+class Q_CORE_EXPORT QDirIterator
+{
 public:
 enum IteratorFlag {
 NoIteratorFlags = 0x0,
diff --git a/src/corelib/io/qfiledevice.cpp b/src/corelib/io/qfiledevice.cpp
index 7fa5c20830d..f9160715048 100644
--- a/src/corelib/io/qfiledevice.cpp
+++ b/src/corelib/io/qfiledevice.cpp
@@ -61,10 +61,9 @@ QFileDevicePrivate::QFileDevicePrivate()
 writeBufferChunkSize = QFILE_WRITEBUFFER_SIZE;
 }

-QFileDevicePrivate::~QFileDevicePrivate()
- = default;
+QFileDevicePrivate::~QFileDevicePrivate() = default;

-QAbstractFileEngine * QFileDevicePrivate::engine() const
+QAbstractFileEngine *QFileDevicePrivate::engine() const
 {
 if (!fileEngine)
 fileEngine = qt_make_unique<QFSFileEngine>();
diff --git a/src/corelib/io/qfsfileengine_p.h b/src/corelib/io/qfsfileengine_p.h
index 639c01571a8..d6e7ecb592e 100644
--- a/src/corelib/io/qfsfileengine_p.h
+++ b/src/corelib/io/qfsfileengine_p.h
@@ -61,7 +61,8 @@

 QT_BEGIN_NAMESPACE

-struct ProcessOpenModeResult {
+struct ProcessOpenModeResult
+{
 bool ok;
 QIODevice::OpenMode openMode;
 QString error;
diff --git a/src/corelib/io/qfsfileengine_win.cpp b/src/corelib/io/qfsfileengine_win.cpp
index 6b1ab5739ed..84ba42f55aa 100644
--- a/src/corelib/io/qfsfileengine_win.cpp
+++ b/src/corelib/io/qfsfileengine_win.cpp
@@ -842,7 +842,7 @@ uchar *QFSFileEnginePrivate::map(qint64 offset, qint64 size,
 NULL);
 // Since this is a special case, we check if the return value was NULL and if so
 // we change it to INVALID_HANDLE_VALUE to follow the logic inside this function.
- if(0 == handle)
+ if (!handle)
 handle = INVALID_HANDLE_VALUE;
 #endif

diff --git a/src/corelib/io/qiodevice.cpp b/src/corelib/io/qiodevice.cpp
index 48f259e3501..5c8c0e6a276 100644
--- a/src/corelib/io/qiodevice.cpp
+++ b/src/corelib/io/qiodevice.cpp
@@ -846,7 +846,7 @@ qint64 QIODevice::pos() const
 */
 qint64 QIODevice::size() const
 {
- return d_func()->isSequential() ? bytesAvailable() : qint64(0);
+ return d_func()->isSequential() ? bytesAvailable() : qint64(0);
 }

 /*!
@@ -1353,7 +1353,7 @@ qint64 QIODevice::readLine(char *data, qint64 maxSize)
 // QRingBuffer::readLine() terminates the line with '\0'
 readSoFar = d->buffer.readLine(data, maxSize + 1);
 if (d->buffer.isEmpty())
- readData(data,0);
+ readData(data, 0);
 if (!sequential)
 d->pos += readSoFar;
 }
diff --git a/src/corelib/io/qiodevice_p.h b/src/corelib/io/qiodevice_p.h
index 8113f08237b..3f8e459d88d 100644
--- a/src/corelib/io/qiodevice_p.h
+++ b/src/corelib/io/qiodevice_p.h
@@ -89,7 +89,8 @@ public:
 qint64 devicePos = 0;
 qint64 transactionPos = 0;

- class QRingBufferRef {
+ class QRingBufferRef
+ {
 QRingBuffer *m_buf;
 inline QRingBufferRef() : m_buf(nullptr) { }
 friend class QIODevicePrivate;
diff --git a/src/corelib/io/qlockfile_win.cpp b/src/corelib/io/qlockfile_win.cpp
index 77cdf666946..46a6b9da5e6 100644
--- a/src/corelib/io/qlockfile_win.cpp
+++ b/src/corelib/io/qlockfile_win.cpp
@@ -167,21 +167,24 @@ QString QLockFilePrivate::processNameByPid(qint64 pid)
 void QLockFile::unlock()
 {
 Q_D(QLockFile);
- if (!d->isLocked)
+ if (!d->isLocked)
 return;
- CloseHandle(d->fileHandle);
- int attempts = 0;
- static const int maxAttempts = 500; // 500ms
- while (!QFile::remove(d->fileName) && ++attempts < maxAttempts) {
- // Someone is reading the lock file right now (on Windows this prevents deleting it).
- QThread::msleep(1);
- }
- if (attempts == maxAttempts) {
- qWarning() << "Could not remove our own lock file" << d->fileName << ". Either other users of the lock file are reading it constantly for 500 ms, or we (no longer) have permissions to delete the file";
- // This is bad because other users of this lock file will now have to wait for the stale-lock-timeout...
- }
- d->lockError = QLockFile::NoError;
- d->isLocked = false;
+ CloseHandle(d->fileHandle);
+ int attempts = 0;
+ static const int maxAttempts = 500; // 500ms
+ while (!QFile::remove(d->fileName) && ++attempts < maxAttempts) {
+ // Someone is reading the lock file right now (on Windows this prevents deleting it).
+ QThread::msleep(1);
+ }
+ if (attempts == maxAttempts) {
+ qWarning() << "Could not remove our own lock file" << d->fileName
+ << ". Either other users of the lock file are reading it constantly for 500 ms, "
+ "or we (no longer) have permissions to delete the file";
+ // This is bad because other users of this lock file will now have to wait for the
+ // stale-lock-timeout...
+ }
+ d->lockError = QLockFile::NoError;
+ d->isLocked = false;
 }

 QT_END_NAMESPACE
diff --git a/src/corelib/io/qloggingregistry_p.h b/src/corelib/io/qloggingregistry_p.h
index 1a924f07dc3..b581af6c2f4 100644
--- a/src/corelib/io/qloggingregistry_p.h
+++ b/src/corelib/io/qloggingregistry_p.h
@@ -74,7 +74,7 @@ public:
 FullText = 0x1,
 LeftFilter = 0x2,
 RightFilter = 0x4,
- MidFilter = LeftFilter | RightFilter
+ MidFilter = LeftFilter | RightFilter
 };
 Q_DECLARE_FLAGS(PatternFlags, PatternFlag)

@@ -144,7 +144,7 @@ private:

 // protected by mutex:
 QList<QLoggingRule> ruleSets[NumRuleSets];
- QHash<QLoggingCategory*,QtMsgType> categories;
+ QHash<QLoggingCategory *, QtMsgType> categories;
 QLoggingCategory::CategoryFilter categoryFilter;

 friend class ::tst_QLoggingRegistry;
diff --git a/src/corelib/io/qnoncontiguousbytedevice.cpp b/src/corelib/io/qnoncontiguousbytedevice.cpp
index df0197e8eb3..cf2da0c8963 100644
--- a/src/corelib/io/qnoncontiguousbytedevice.cpp
+++ b/src/corelib/io/qnoncontiguousbytedevice.cpp
@@ -297,7 +297,7 @@ QNonContiguousByteDeviceIoDeviceImpl::~QNonContiguousByteDeviceIoDeviceImpl()
 delete currentReadBuffer;
 }

-const char* QNonContiguousByteDeviceIoDeviceImpl::readPointer(qint64 maximumLength, qint64 &len)
+const char *QNonContiguousByteDeviceIoDeviceImpl::readPointer(qint64 maximumLength, qint64 &len)
 {
 if (eof == true) {
 len = -1;
@@ -360,7 +360,6 @@ bool QNonContiguousByteDeviceIoDeviceImpl::advanceReadPointer(qint64 amount)
 currentReadBufferAmount = 0;
 }

-
 return true;
 }

@@ -374,7 +373,7 @@ bool QNonContiguousByteDeviceIoDeviceImpl::reset()
 bool reset = (initialPosition == 0) ? device->reset() : device->seek(initialPosition);
 if (reset) {
 eof = false; // assume eof is false, it will be true after a read has been attempted
- totalAdvancements = 0; //reset the progress counter
+ totalAdvancements = 0; // reset the progress counter
 if (currentReadBuffer) {
 delete currentReadBuffer;
 currentReadBuffer = nullptr;
@@ -441,8 +440,7 @@ qint64 QByteDeviceWrappingIoDevice::size() const
 return byteDevice->size();
 }

-
-qint64 QByteDeviceWrappingIoDevice::readData( char * data, qint64 maxSize)
+qint64 QByteDeviceWrappingIoDevice::readData(char *data, qint64 maxSize)
 {
 qint64 len;
 const char *readPointer = byteDevice->readPointer(maxSize, len);
@@ -454,7 +452,7 @@ qint64 QByteDeviceWrappingIoDevice::readData( char * data, qint64 maxSize)
 return len;
 }

-qint64 QByteDeviceWrappingIoDevice::writeData( const char* data, qint64 maxSize)
+qint64 QByteDeviceWrappingIoDevice::writeData(const char *data, qint64 maxSize)
 {
 Q_UNUSED(data);
 Q_UNUSED(maxSize);
@@ -482,10 +480,10 @@ qint64 QByteDeviceWrappingIoDevice::writeData( const char* data, qint64 maxSize)

 \internal
 */
-QNonContiguousByteDevice* QNonContiguousByteDeviceFactory::create(QIODevice *device)
+QNonContiguousByteDevice *QNonContiguousByteDeviceFactory::create(QIODevice *device)
 {
 // shortcut if it is a QBuffer
- if (QBuffer* buffer = qobject_cast<QBuffer*>(device)) {
+ if (QBuffer *buffer = qobject_cast<QBuffer *>(device)) {
 return new QNonContiguousByteDeviceBufferImpl(buffer);
 }

@@ -566,7 +564,7 @@ QSharedPointer<QNonContiguousByteDevice> QNonContiguousByteDeviceFactory::create

 \internal
 */
-QIODevice* QNonContiguousByteDeviceFactory::wrap(QNonContiguousByteDevice* byteDevice)
+QIODevice *QNonContiguousByteDeviceFactory::wrap(QNonContiguousByteDevice *byteDevice)
 {
 // ### FIXME if it already has been based on QIoDevice, we could that one out again
 // and save some calling
diff --git a/src/corelib/io/qnoncontiguousbytedevice_p.h b/src/corelib/io/qnoncontiguousbytedevice_p.h
index bbc4ea5ae2f..aa762766afc 100644
--- a/src/corelib/io/qnoncontiguousbytedevice_p.h
+++ b/src/corelib/io/qnoncontiguousbytedevice_p.h
@@ -64,7 +64,7 @@ class Q_CORE_EXPORT QNonContiguousByteDevice : public QObject
 {
 Q_OBJECT
 public:
- virtual const char* readPointer(qint64 maximumLength, qint64 &len) = 0;
+ virtual const char *readPointer(qint64 maximumLength, qint64 &len) = 0;
 virtual bool advanceReadPointer(qint64 amount) = 0;
 virtual bool atEnd() const = 0;
 virtual qint64 pos() const { return -1; }
@@ -76,7 +76,6 @@ public:
 protected:
 QNonContiguousByteDevice();

-
 Q_SIGNALS:
 void readyRead();
 void readProgress(qint64 current, qint64 total);
@@ -85,16 +84,16 @@ Q_SIGNALS:
 class Q_CORE_EXPORT QNonContiguousByteDeviceFactory
 {
 public:
- static QNonContiguousByteDevice* create(QIODevice *device);
+ static QNonContiguousByteDevice *create(QIODevice *device);
 static QSharedPointer<QNonContiguousByteDevice> createShared(QIODevice *device);

- static QNonContiguousByteDevice* create(QByteArray *byteArray);
+ static QNonContiguousByteDevice *create(QByteArray *byteArray);
 static QSharedPointer<QNonContiguousByteDevice> createShared(QByteArray *byteArray);

- static QNonContiguousByteDevice* create(QSharedPointer<QRingBuffer> ringBuffer);
+ static QNonContiguousByteDevice *create(QSharedPointer<QRingBuffer> ringBuffer);
 static QSharedPointer<QNonContiguousByteDevice> createShared(QSharedPointer<QRingBuffer> ringBuffer);

- static QIODevice* wrap(QNonContiguousByteDevice* byteDevice);
+ static QIODevice *wrap(QNonContiguousByteDevice *byteDevice);
 };

 // the actual implementations
@@ -105,14 +104,15 @@ class QNonContiguousByteDeviceByteArrayImpl : public QNonContiguousByteDevice
 public:
 QNonContiguousByteDeviceByteArrayImpl(QByteArray *ba);
 ~QNonContiguousByteDeviceByteArrayImpl();
- const char* readPointer(qint64 maximumLength, qint64 &len) override;
+ const char *readPointer(qint64 maximumLength, qint64 &len) override;
 bool advanceReadPointer(qint64 amount) override;
 bool atEnd() const override;
 bool reset() override;
 qint64 size() const override;
 qint64 pos() const override;
+
 protected:
- QByteArray* byteArray;
+ QByteArray *byteArray;
 qint64 currentPosition;
 };

@@ -121,33 +121,34 @@ class QNonContiguousByteDeviceRingBufferImpl : public QNonContiguousByteDevice
 public:
 QNonContiguousByteDeviceRingBufferImpl(QSharedPointer<QRingBuffer> rb);
 ~QNonContiguousByteDeviceRingBufferImpl();
- const char* readPointer(qint64 maximumLength, qint64 &len) override;
+ const char *readPointer(qint64 maximumLength, qint64 &len) override;
 bool advanceReadPointer(qint64 amount) override;
 bool atEnd() const override;
 bool reset() override;
 qint64 size() const override;
 qint64 pos() const override;
+
 protected:
 QSharedPointer<QRingBuffer> ringBuffer;
 qint64 currentPosition;
 };

-
 class QNonContiguousByteDeviceIoDeviceImpl : public QNonContiguousByteDevice
 {
 Q_OBJECT
 public:
 QNonContiguousByteDeviceIoDeviceImpl(QIODevice *d);
 ~QNonContiguousByteDeviceIoDeviceImpl();
- const char* readPointer(qint64 maximumLength, qint64 &len) override;
+ const char *readPointer(qint64 maximumLength, qint64 &len) override;
 bool advanceReadPointer(qint64 amount) override;
 bool atEnd() const override;
 bool reset() override;
 qint64 size() const override;
 qint64 pos() const override;
+
 protected:
- QIODevice* device;
- QByteArray* currentReadBuffer;
+ QIODevice *device;
+ QByteArray *currentReadBuffer;
 qint64 currentReadBufferSize;
 qint64 currentReadBufferAmount;
 qint64 currentReadBufferPosition;
@@ -162,32 +163,34 @@ class QNonContiguousByteDeviceBufferImpl : public QNonContiguousByteDevice
 public:
 QNonContiguousByteDeviceBufferImpl(QBuffer *b);
 ~QNonContiguousByteDeviceBufferImpl();
- const char* readPointer(qint64 maximumLength, qint64 &len) override;
+ const char *readPointer(qint64 maximumLength, qint64 &len) override;
 bool advanceReadPointer(qint64 amount) override;
 bool atEnd() const override;
 bool reset() override;
 qint64 size() const override;
+
 protected:
- QBuffer* buffer;
+ QBuffer *buffer;
 QByteArray byteArray;
- QNonContiguousByteDeviceByteArrayImpl* arrayImpl;
+ QNonContiguousByteDeviceByteArrayImpl *arrayImpl;
 };

 // ... and the reverse thing
 class QByteDeviceWrappingIoDevice : public QIODevice
 {
 public:
- QByteDeviceWrappingIoDevice (QNonContiguousByteDevice *bd);
- ~QByteDeviceWrappingIoDevice ();
+ QByteDeviceWrappingIoDevice(QNonContiguousByteDevice *bd);
+ ~QByteDeviceWrappingIoDevice();
 bool isSequential() const override;
 bool atEnd() const override;
 bool reset() override;
 qint64 size() const override;
+
 protected:
 qint64 readData(char *data, qint64 maxSize) override;
 qint64 writeData(const char *data, qint64 maxSize) override;

- QNonContiguousByteDevice *byteDevice;
+ QNonContiguousByteDevice *byteDevice;
 };

 QT_END_NAMESPACE
diff --git a/src/corelib/io/qresource.cpp b/src/corelib/io/qresource.cpp
index 80247de9a0d..aa144ecadda 100644
--- a/src/corelib/io/qresource.cpp
+++ b/src/corelib/io/qresource.cpp
@@ -107,13 +107,15 @@ public:
 {
 }

- inline bool hasNext() {
+ inline bool hasNext()
+ {
 while (m_pos < m_len && m_data[m_pos] == m_splitChar)
 ++m_pos;
 return m_pos < m_len;
 }

- inline QStringView next() {
+ inline QStringView next()
+ {
 int start = m_pos;
 while (m_pos < m_len && m_data[m_pos] != m_splitChar)
 ++m_pos;
@@ -126,18 +128,17 @@ public:
 QChar m_splitChar = QLatin1Char('/');
 };

-
-//resource glue
+// resource glue
 class QResourceRoot
 {
 public:
- enum Flags
- {
+ enum Flags {
 // must match rcc.h
 Compressed = 0x01,
 Directory = 0x02,
 CompressedZstd = 0x04
 };
+
 private:
 const uchar *tree, *names, *payloads;
 int version;
@@ -305,7 +306,7 @@ public:

 QLocale locale;
 QString fileName, absoluteFilePath;
- QList<QResourceRoot*> related;
+ QList<QResourceRoot *> related;
 mutable qint64 size;
 mutable quint64 lastModified;
 mutable const uchar *data;
@@ -318,8 +319,7 @@ public:
 Q_DECLARE_PUBLIC(QResource)
 };

-void
-QResourcePrivate::clear()
+void QResourcePrivate::clear()
 {
 absoluteFilePath.clear();
 compressionAlgo = QResource::NoCompression;
@@ -328,28 +328,27 @@ QResourcePrivate::clear()
 children.clear();
 lastModified = 0;
 container = 0;
- for(int i = 0; i < related.size(); ++i) {
+ for (int i = 0; i < related.size(); ++i) {
 QResourceRoot *root = related.at(i);
- if(!root->ref.deref())
+ if (!root->ref.deref())
 delete root;
 }
 related.clear();
 }

-bool
-QResourcePrivate::load(const QString &file)
+bool QResourcePrivate::load(const QString &file)
 {
 related.clear();
 const auto locker = qt_scoped_lock(resourceMutex());
 const ResourceList *list = resourceList();
 QString cleaned = cleanPath(file);
- for(int i = 0; i < list->size(); ++i) {
+ for (int i = 0; i < list->size(); ++i) {
 QResourceRoot *res = list->at(i);
 const int node = res->findNode(cleaned, locale);
- if(node != -1) {
- if(related.isEmpty()) {
+ if (node != -1) {
+ if (related.isEmpty()) {
 container = res->isContainer(node);
- if(!container) {
+ if (!container) {
 data = res->data(node, &size);
 compressionAlgo = res->compressionAlgo(node);
 } else {
@@ -358,12 +357,13 @@ QResourcePrivate::load(const QString &file)
 compressionAlgo = QResource::NoCompression;
 }
 lastModified = res->lastModified(node);
- } else if(res->isContainer(node) != container) {
- qWarning("QResourceInfo: Resource [%s] has both data and children!", file.toLatin1().constData());
+ } else if (res->isContainer(node) != container) {
+ qWarning("QResourceInfo: Resource [%s] has both data and children!",
+ file.toLatin1().constData());
 }
 res->ref.ref();
 related.append(res);
- } else if(res->mappingRootSubdir(file)) {
+ } else if (res->mappingRootSubdir(file)) {
 container = true;
 data = nullptr;
 size = 0;
@@ -376,31 +376,30 @@ QResourcePrivate::load(const QString &file)
 return !related.isEmpty();
 }

-void
-QResourcePrivate::ensureInitialized() const
+void QResourcePrivate::ensureInitialized() const
 {
- if(!related.isEmpty())
+ if (!related.isEmpty())
 return;
 QResourcePrivate *that = const_cast<QResourcePrivate *>(this);
- if(fileName == QLatin1String(":"))
+ if (fileName == QLatin1String(":"))
 that->fileName += QLatin1Char('/');
 that->absoluteFilePath = fileName;
- if(!that->absoluteFilePath.startsWith(QLatin1Char(':')))
+ if (!that->absoluteFilePath.startsWith(QLatin1Char(':')))
 that->absoluteFilePath.prepend(QLatin1Char(':'));

 QStringView path(fileName);
- if(path.startsWith(QLatin1Char(':')))
+ if (path.startsWith(QLatin1Char(':')))
 path = path.mid(1);

- if(path.startsWith(QLatin1Char('/'))) {
+ if (path.startsWith(QLatin1Char('/'))) {
 that->load(path.toString());
 } else {
 const auto locker = qt_scoped_lock(resourceMutex());
 QStringList searchPaths = *resourceSearchPaths();
 searchPaths << QLatin1String("");
- for(int i = 0; i < searchPaths.size(); ++i) {
+ for (int i = 0; i < searchPaths.size(); ++i) {
 const QString searchPath(searchPaths.at(i) + QLatin1Char('/') + path);
- if(that->load(searchPath)) {
+ if (that->load(searchPath)) {
 that->absoluteFilePath = QLatin1Char(':') + searchPath;
 break;
 }
@@ -408,29 +407,28 @@ QResourcePrivate::ensureInitialized() const
 }
 }

-void
-QResourcePrivate::ensureChildren() const
+void QResourcePrivate::ensureChildren() const
 {
 ensureInitialized();
- if(!children.isEmpty() || !container || related.isEmpty())
+ if (!children.isEmpty() || !container || related.isEmpty())
 return;

 QString path = absoluteFilePath, k;
- if(path.startsWith(QLatin1Char(':')))
+ if (path.startsWith(QLatin1Char(':')))
 path = path.mid(1);
 QDuplicateTracker<QString> kids;
 kids.reserve(related.size());
 QString cleaned = cleanPath(path);
- for(int i = 0; i < related.size(); ++i) {
+ for (int i = 0; i < related.size(); ++i) {
 QResourceRoot *res = related.at(i);
- if(res->mappingRootSubdir(path, &k) && !k.isEmpty()) {
+ if (res->mappingRootSubdir(path, &k) && !k.isEmpty()) {
 if (!kids.hasSeen(k))
 children += k;
 } else {
 const int node = res->findNode(cleaned);
- if(node != -1) {
+ if (node != -1) {
 QStringList related_children = res->children(node);
- for(int kid = 0; kid < related_children.size(); ++kid) {
+ for (int kid = 0; kid < related_children.size(); ++kid) {
 k = related_children.at(kid);
 if (!kids.hasSeen(k))
 children += k;
@@ -466,7 +464,6 @@ qint64 QResourcePrivate::uncompressedSize() const
 Q_UNREACHABLE();
 #endif
 }
-
 }
 return -1;
 }
@@ -483,8 +480,8 @@ qsizetype QResourcePrivate::decompress(char *buffer, qsizetype bufferSize) const
 case QResource::ZlibCompression: {
 #ifndef QT_NO_COMPRESS
 uLong len = uLong(bufferSize);
- int res = ::uncompress(reinterpret_cast<Bytef *>(buffer), &len,
- data + sizeof(quint32), uLong(size - sizeof(quint32)));
+ int res = ::uncompress(reinterpret_cast<Bytef *>(buffer), &len, data + sizeof(quint32),
+ uLong(size - sizeof(quint32)));
 if (res != Z_OK) {
 qWarning("QResource: error decompressing zlib content (%d)", res);
 return -1;
@@ -776,16 +773,16 @@ QStringList QResource::children() const

 inline uint QResourceRoot::hash(int node) const
 {
- if(!node) //root
+ if (!node) // root
 return 0;
 const int offset = findOffset(node);
 qint32 name_offset = qFromBigEndian<qint32>(tree + offset);
- name_offset += 2; //jump past name length
+ name_offset += 2; // jump past name length
 return qFromBigEndian<quint32>(names + name_offset);
 }
 inline QString QResourceRoot::name(int node) const
 {
- if(!node) // root
+ if (!node) // root
 return QString();
 const int offset = findOffset(node);

@@ -793,7 +790,7 @@ inline QString QResourceRoot::name(int node) const
 qint32 name_offset = qFromBigEndian<qint32>(tree + offset);
 quint16 name_length = qFromBigEndian<qint16>(names + name_offset);
 name_offset += 2;
- name_offset += 4; //jump past hash
+ name_offset += 4; // jump past hash

 ret.resize(name_length);
 QChar *strData = ret.data();
@@ -806,15 +803,15 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 QString path = _path;
 {
 QString root = mappingRoot();
- if(!root.isEmpty()) {
- if(root == path) {
+ if (!root.isEmpty()) {
+ if (root == path) {
 path = QLatin1Char('/');
 } else {
- if(!root.endsWith(QLatin1Char('/')))
+ if (!root.endsWith(QLatin1Char('/')))
 root += QLatin1Char('/');
- if(path.size() >= root.size() && path.startsWith(root))
- path = path.mid(root.length()-1);
- if(path.isEmpty())
+ if (path.size() >= root.size() && path.startsWith(root))
+ path = path.mid(root.length() - 1);
+ if (path.isEmpty())
 path = QLatin1Char('/');
 }
 }
@@ -823,14 +820,14 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 qDebug() << "!!!!" << "START" << path << locale.country() << locale.language();
 #endif

- if(path == QLatin1String("/"))
+ if (path == QLatin1String("/"))
 return 0;

- //the root node is always first
+ // the root node is always first
 qint32 child_count = qFromBigEndian<qint32>(tree + 6);
 qint32 child = qFromBigEndian<qint32>(tree + 10);

- //now iterate up the tree
+ // now iterate up the tree
 int node = -1;

 QStringSplitter splitter(path);
@@ -839,20 +836,20 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const

 #ifdef DEBUG_RESOURCE_MATCH
 qDebug() << " CHILDREN" << segment;
- for(int j = 0; j < child_count; ++j) {
- qDebug() << " " << child+j << " :: " << name(child+j);
+ for (int j = 0; j < child_count; ++j) {
+ qDebug() << " " << child + j << " :: " << name(child + j);
 }
 #endif
 const uint h = qt_hash(segment);

- //do the binary search for the hash
- int l = 0, r = child_count-1;
- int sub_node = (l+r+1)/2;
- while(r != l) {
- const uint sub_node_hash = hash(child+sub_node);
- if(h == sub_node_hash)
+ // do the binary search for the hash
+ int l = 0, r = child_count - 1;
+ int sub_node = (l + r + 1) / 2;
+ while (r != l) {
+ const uint sub_node_hash = hash(child + sub_node);
+ if (h == sub_node_hash)
 break;
- else if(h < sub_node_hash)
+ else if (h < sub_node_hash)
 r = sub_node - 1;
 else
 l = sub_node;
@@ -860,25 +857,26 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 }
 sub_node += child;

- //now do the "harder" compares
+ // now do the "harder" compares
 bool found = false;
- if(hash(sub_node) == h) {
- while(sub_node > child && hash(sub_node-1) == h) //backup for collisions
+ if (hash(sub_node) == h) {
+ while (sub_node > child && hash(sub_node - 1) == h) // backup for collisions
 --sub_node;
- for(; sub_node < child+child_count && hash(sub_node) == h; ++sub_node) { //here we go...
- if(name(sub_node) == segment) {
+ for (; sub_node < child + child_count && hash(sub_node) == h;
+ ++sub_node) { // here we go...
+ if (name(sub_node) == segment) {
 found = true;
 int offset = findOffset(sub_node);
 #ifdef DEBUG_RESOURCE_MATCH
 qDebug() << " TRY" << sub_node << name(sub_node) << offset;
 #endif
- offset += 4; //jump past name
+ offset += 4; // jump past name

 const qint16 flags = qFromBigEndian<qint16>(tree + offset);
 offset += 2;

- if(!splitter.hasNext()) {
- if(!(flags & Directory)) {
+ if (!splitter.hasNext()) {
+ if (!(flags & Directory)) {
 const qint16 country = qFromBigEndian<qint16>(tree + offset);
 offset += 2;

@@ -887,13 +885,15 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 #ifdef DEBUG_RESOURCE_MATCH
 qDebug() << " " << "LOCALE" << country << language;
 #endif
- if(country == locale.country() && language == locale.language()) {
+ if (country == locale.country() && language == locale.language()) {
 #ifdef DEBUG_RESOURCE_MATCH
 qDebug() << "!!!!" << "FINISHED" << __LINE__ << sub_node;
 #endif
 return sub_node;
- } else if((country == QLocale::AnyCountry && language == locale.language()) ||
- (country == QLocale::AnyCountry && language == QLocale::C && node == -1)) {
+ } else if ((country == QLocale::AnyCountry
+ && language == locale.language())
+ || (country == QLocale::AnyCountry && language == QLocale::C
+ && node == -1)) {
 node = sub_node;
 }
 continue;
@@ -906,7 +906,7 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 }
 }

- if(!(flags & Directory))
+ if (!(flags & Directory))
 return -1;

 child_count = qFromBigEndian<qint32>(tree + offset);
@@ -916,7 +916,7 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 }
 }
 }
- if(!found)
+ if (!found)
 break;
 }
 #ifdef DEBUG_RESOURCE_MATCH
@@ -926,28 +926,28 @@ int QResourceRoot::findNode(const QString &_path, const QLocale &locale) const
 }
 short QResourceRoot::flags(int node) const
 {
- if(node == -1)
+ if (node == -1)
 return 0;
- const int offset = findOffset(node) + 4; //jump past name
+ const int offset = findOffset(node) + 4; // jump past name
 return qFromBigEndian<qint16>(tree + offset);
 }
 const uchar *QResourceRoot::data(int node, qint64 *size) const
 {
- if(node == -1) {
+ if (node == -1) {
 *size = 0;
 return nullptr;
 }
- int offset = findOffset(node) + 4; //jump past name
+ int offset = findOffset(node) + 4; // jump past name

 const qint16 flags = qFromBigEndian<qint16>(tree + offset);
 offset += 2;

- offset += 4; //jump past locale
+ offset += 4; // jump past locale

- if(!(flags & Directory)) {
+ if (!(flags & Directory)) {
 const qint32 data_offset = qFromBigEndian<qint32>(tree + offset);
 const quint32 data_length = qFromBigEndian<quint32>(payloads + data_offset);
- const uchar *ret = payloads+data_offset+4;
+ const uchar *ret = payloads + data_offset + 4;
 *size = data_length;
 return ret;
 }
@@ -967,20 +967,20 @@ quint64 QResourceRoot::lastModified(int node) const

 QStringList QResourceRoot::children(int node) const
 {
- if(node == -1)
+ if (node == -1)
 return QStringList();
- int offset = findOffset(node) + 4; //jump past name
+ int offset = findOffset(node) + 4; // jump past name

 const qint16 flags = qFromBigEndian<qint16>(tree + offset);
 offset += 2;

 QStringList ret;
- if(flags & Directory) {
+ if (flags & Directory) {
 const qint32 child_count = qFromBigEndian<qint32>(tree + offset);
 offset += 4;
 const qint32 child_off = qFromBigEndian<qint32>(tree + offset);
 ret.reserve(child_count);
- for(int i = child_off; i < child_off+child_count; ++i)
+ for (int i = child_off; i < child_off + child_count; ++i)
 ret << name(i);
 }
 return ret;
@@ -1024,7 +1024,7 @@ Q_CORE_EXPORT bool qRegisterResourceData(int version, const unsigned char *tree,
 break;
 }
 }
- if(!found) {
+ if (!found) {
 QResourceRoot *root = new QResourceRoot(version, tree, name, data);
 root->ref.ref();
 list->append(root);
@@ -1044,10 +1044,10 @@ Q_CORE_EXPORT bool qUnregisterResourceData(int version, const unsigned char *tre
 if (version >= 0x01 && version <= 0x3) {
 QResourceRoot res(version, tree, name, data);
 ResourceList *list = resourceList();
- for (int i = 0; i < list->size(); ) {
+ for (int i = 0; i < list->size();) {
 if (*list->at(i) == res) {
 QResourceRoot *root = list->takeAt(i);
- if(!root->ref.deref())
+ if (!root->ref.deref())
 delete root;
 } else {
 ++i;
@@ -1058,9 +1058,9 @@ Q_CORE_EXPORT bool qUnregisterResourceData(int version, const unsigned char *tre
 return false;
 }

-//run time resource creation
+// run time resource creation

-class QDynamicBufferResourceRoot: public QResourceRoot
+class QDynamicBufferResourceRoot : public QResourceRoot
 {
 QString root;
 const uchar *buffer;
@@ -1079,12 +1079,12 @@ public:
 if (size >= 0 && size < 20)
 return false;

- //setup the data now
+ // setup the data now
 int offset = 0;

- //magic number
- if(b[offset+0] != 'q' || b[offset+1] != 'r' ||
- b[offset+2] != 'e' || b[offset+3] != 's') {
+ // magic number
+ if (b[offset + 0] != 'q' || b[offset + 1] != 'r' || b[offset + 2] != 'e'
+ || b[offset + 3] != 's') {
 return false;
 }
 offset += 4;
@@ -1123,14 +1123,14 @@ public:

 if (version >= 0x01 && version <= 0x03) {
 buffer = b;
- setSource(version, b+tree_offset, b+name_offset, b+data_offset);
+ setSource(version, b + tree_offset, b + name_offset, b + data_offset);
 return true;
 }
 return false;
 }
 };

-class QDynamicFileResourceRoot: public QDynamicBufferResourceRoot
+class QDynamicFileResourceRoot : public QDynamicBufferResourceRoot
 {
 QString fileName;
 // for mmap'ed files, this is what needs to be unmapped.
@@ -1144,13 +1144,13 @@ public:
 ~QDynamicFileResourceRoot() {
 #if defined(QT_USE_MMAP)
 if (unmapPointer) {
- munmap((char*)unmapPointer, unmapLength);
+ munmap(reinterpret_cast<char *>(unmapPointer), unmapLength);
 unmapPointer = nullptr;
 unmapLength = 0;
 } else
 #endif
 {
- delete [] mappingBuffer();
+ delete[] mappingBuffer();
 }
 }
 QString mappingFile() const { return fileName; }
@@ -1199,11 +1199,11 @@ bool QDynamicFileResourceRoot::registerSelf(const QString &f)
 if (fsize <= std::numeric_limits<qsizetype>::max()) {
 data_len = file.size();
 data = new uchar[data_len];
- ok = (data_len == file.read((char*)data, data_len));
+ ok = (data_len == file.read(reinterpret_cast<char *>(data), data_len));
 }
 }
 if (!ok) {
- delete [] data;
+ delete[] data;
 data = nullptr;
 data_len = 0;
 return false;
@@ -1221,17 +1221,17 @@ bool QDynamicFileResourceRoot::registerSelf(const QString &f)
 return false;
 }

-static QString qt_resource_fixResourceRoot(QString r) {
- if(!r.isEmpty()) {
- if(r.startsWith(QLatin1Char(':')))
+static QString qt_resource_fixResourceRoot(QString r)
+{
+ if (!r.isEmpty()) {
+ if (r.startsWith(QLatin1Char(':')))
 r = r.mid(1);
- if(!r.isEmpty())
+ if (!r.isEmpty())
 r = QDir::cleanPath(r);
 }
 return r;
 }

-
 /*!
 \fn bool QResource::registerResource(const QString &rccFileName, const QString &mapRoot)

@@ -1242,18 +1242,18 @@ static QString qt_resource_fixResourceRoot(QString r) {
 \sa unregisterResource()
 */

-bool
-QResource::registerResource(const QString &rccFilename, const QString &resourceRoot)
+bool QResource::registerResource(const QString &rccFilename, const QString &resourceRoot)
 {
 QString r = qt_resource_fixResourceRoot(resourceRoot);
- if(!r.isEmpty() && r[0] != QLatin1Char('/')) {
- qWarning("QDir::registerResource: Registering a resource [%ls] must be rooted in an absolute path (start with /) [%ls]",
+ if (!r.isEmpty() && r[0] != QLatin1Char('/')) {
+ qWarning("QDir::registerResource: Registering a resource [%ls] must be rooted in an "
+ "absolute path (start with /) [%ls]",
 qUtf16Printable(rccFilename), qUtf16Printable(resourceRoot));
 return false;
 }

 QDynamicFileResourceRoot *root = new QDynamicFileResourceRoot(r);
- if(root->registerSelf(rccFilename)) {
+ if (root->registerSelf(rccFilename)) {
 root->ref.ref();
 const auto locker = qt_scoped_lock(resourceMutex());
 resourceList()->append(root);
@@ -1274,20 +1274,19 @@ QResource::registerResource(const QString &rccFilename, const QString &resourceR
 \sa registerResource()
 */

-bool
-QResource::unregisterResource(const QString &rccFilename, const QString &resourceRoot)
+bool QResource::unregisterResource(const QString &rccFilename, const QString &resourceRoot)
 {
 QString r = qt_resource_fixResourceRoot(resourceRoot);

 const auto locker = qt_scoped_lock(resourceMutex());
 ResourceList *list = resourceList();
- for(int i = 0; i < list->size(); ++i) {
+ for (int i = 0; i < list->size(); ++i) {
 QResourceRoot *res = list->at(i);
- if(res->type() == QResourceRoot::Resource_File) {
- QDynamicFileResourceRoot *root = reinterpret_cast<QDynamicFileResourceRoot*>(res);
+ if (res->type() == QResourceRoot::Resource_File) {
+ QDynamicFileResourceRoot *root = reinterpret_cast<QDynamicFileResourceRoot *>(res);
 if (root->mappingFile() == rccFilename && root->mappingRoot() == r) {
 list->removeAt(i);
- if(!root->ref.deref()) {
+ if (!root->ref.deref()) {
 delete root;
 return true;
 }
@@ -1298,7 +1297,6 @@ QResource::unregisterResource(const QString &rccFilename, const QString &resourc
 return false;
 }

-
 /*!
 \fn bool QResource::registerResource(const uchar *rccData, const QString &mapRoot)
 \since 4.3
@@ -1313,12 +1311,12 @@ QResource::unregisterResource(const QString &rccFilename, const QString &resourc
 \sa unregisterResource()
 */

-bool
-QResource::registerResource(const uchar *rccData, const QString &resourceRoot)
+bool QResource::registerResource(const uchar *rccData, const QString &resourceRoot)
 {
 QString r = qt_resource_fixResourceRoot(resourceRoot);
- if(!r.isEmpty() && r[0] != QLatin1Char('/')) {
- qWarning("QDir::registerResource: Registering a resource [%p] must be rooted in an absolute path (start with /) [%ls]",
+ if (!r.isEmpty() && r[0] != QLatin1Char('/')) {
+ qWarning("QDir::registerResource: Registering a resource [%p] must be rooted in an "
+ "absolute path (start with /) [%ls]",
 rccData, qUtf16Printable(resourceRoot));
 return false;
 }
@@ -1345,20 +1343,19 @@ QResource::registerResource(const uchar *rccData, const QString &resourceRoot)
 \sa registerResource()
 */

-bool
-QResource::unregisterResource(const uchar *rccData, const QString &resourceRoot)
+bool QResource::unregisterResource(const uchar *rccData, const QString &resourceRoot)
 {
 QString r = qt_resource_fixResourceRoot(resourceRoot);

 const auto locker = qt_scoped_lock(resourceMutex());
 ResourceList *list = resourceList();
- for(int i = 0; i < list->size(); ++i) {
+ for (int i = 0; i < list->size(); ++i) {
 QResourceRoot *res = list->at(i);
- if(res->type() == QResourceRoot::Resource_Buffer) {
- QDynamicBufferResourceRoot *root = reinterpret_cast<QDynamicBufferResourceRoot*>(res);
+ if (res->type() == QResourceRoot::Resource_Buffer) {
+ QDynamicBufferResourceRoot *root = reinterpret_cast<QDynamicBufferResourceRoot *>(res);
 if (root->mappingBuffer() == rccData && root->mappingRoot() == r) {
 list->removeAt(i);
- if(!root->ref.deref()) {
+ if (!root->ref.deref()) {
 delete root;
 return true;
 }
@@ -1370,7 +1367,7 @@ QResource::unregisterResource(const uchar *rccData, const QString &resourceRoot)
 }

 #if !defined(QT_BOOTSTRAPPED)
-//resource engine
+// resource engine
 class QResourceFileEnginePrivate : public QAbstractFileEnginePrivate
 {
 protected:
@@ -1466,14 +1463,14 @@ bool QResourceFileEngine::flush()
 qint64 QResourceFileEngine::read(char *data, qint64 len)
 {
 Q_D(QResourceFileEngine);
- if(len > size()-d->offset)
- len = size()-d->offset;
- if(len <= 0)
+ if (len > size() - d->offset)
+ len = size() - d->offset;
+ if (len <= 0)
 return 0;
 if (!d->uncompressed.isNull())
- memcpy(data, d->uncompressed.constData()+d->offset, len);
+ memcpy(data, d->uncompressed.constData() + d->offset, len);
 else
- memcpy(data, d->resource.data()+d->offset, len);
+ memcpy(data, d->resource.data() + d->offset, len);
 d->offset += len;
 return len;
 }
@@ -1518,7 +1515,7 @@ qint64 QResourceFileEngine::pos() const
 bool QResourceFileEngine::atEnd() const
 {
 Q_D(const QResourceFileEngine);
- if(!d->resource.isValid())
+ if (!d->resource.isValid())
 return true;
 return d->offset == size();
 }
@@ -1526,10 +1523,10 @@ bool QResourceFileEngine::atEnd() const
 bool QResourceFileEngine::seek(qint64 pos)
 {
 Q_D(QResourceFileEngine);
- if(!d->resource.isValid())
+ if (!d->resource.isValid())
 return false;

- if(d->offset > size())
+ if (d->offset > size())
 return false;
 d->offset = pos;
 return true;
@@ -1544,20 +1541,21 @@ QAbstractFileEngine::FileFlags QResourceFileEngine::fileFlags(QAbstractFileEngin
 {
 Q_D(const QResourceFileEngine);
 QAbstractFileEngine::FileFlags ret;
- if(!d->resource.isValid())
+ if (!d->resource.isValid())
 return ret;

- if(type & PermsMask)
- ret |= QAbstractFileEngine::FileFlags(ReadOwnerPerm|ReadUserPerm|ReadGroupPerm|ReadOtherPerm);
- if(type & TypesMask) {
- if(d->resource.isDir())
+ if (type & PermsMask)
+ ret |= QAbstractFileEngine::FileFlags(ReadOwnerPerm | ReadUserPerm | ReadGroupPerm
+ | ReadOtherPerm);
+ if (type & TypesMask) {
+ if (d->resource.isDir())
 ret |= DirectoryType;
 else
 ret |= FileType;
 }
- if(type & FlagsMask) {
+ if (type & FlagsMask) {
 ret |= ExistsFlag;
- if(d->resource.absoluteFilePath() == QLatin1String(":/"))
+ if (d->resource.absoluteFilePath() == QLatin1String(":/"))
 ret |= RootFlag;
 }
 return ret;
@@ -1571,13 +1569,14 @@ bool QResourceFileEngine::setPermissions(uint)
 QString QResourceFileEngine::fileName(FileName file) const
 {
 Q_D(const QResourceFileEngine);
- if(file == BaseName) {
+ if (file == BaseName) {
 int slash = d->resource.fileName().lastIndexOf(QLatin1Char('/'));
 if (slash == -1)
 return d->resource.fileName();
 return d->resource.fileName().mid(slash + 1);
- } else if(file == PathName || file == AbsolutePathName) {
- const QString path = (file == AbsolutePathName) ? d->resource.absoluteFilePath() : d->resource.fileName();
+ } else if (file == PathName || file == AbsolutePathName) {
+ const QString path = (file == AbsolutePathName) ? d->resource.absoluteFilePath()
+ : d->resource.fileName();
 const int slash = path.lastIndexOf(QLatin1Char('/'));
 if (slash == -1)
 return QLatin1String(":");
@@ -1585,9 +1584,9 @@ QString QResourceFileEngine::fileName(FileName file) const
 return QLatin1String(":/");
 return path.left(slash);

- } else if(file == CanonicalName || file == CanonicalPathName) {
+ } else if (file == CanonicalName || file == CanonicalPathName) {
 const QString absoluteFilePath = d->resource.absoluteFilePath();
- if(file == CanonicalPathName) {
+ if (file == CanonicalPathName) {
 const int slash = absoluteFilePath.lastIndexOf(QLatin1Char('/'));
 if (slash != -1)
 return absoluteFilePath.left(slash);
@@ -1604,7 +1603,7 @@ bool QResourceFileEngine::isRelativePath() const

 uint QResourceFileEngine::ownerId(FileOwner) const
 {
- static const uint nobodyID = (uint) -2;
+ static const uint nobodyID = static_cast<uint>(-2);
 return nobodyID;
 }

@@ -1642,13 +1641,13 @@ bool QResourceFileEngine::extension(Extension extension, const ExtensionOption *
 {
 Q_D(QResourceFileEngine);
 if (extension == MapExtension) {
- const MapExtensionOption *options = (const MapExtensionOption*)(option);
- MapExtensionReturn *returnValue = static_cast<MapExtensionReturn*>(output);
+ const auto *options = static_cast<const MapExtensionOption *>(option);
+ auto *returnValue = static_cast<MapExtensionReturn *>(output);
 returnValue->address = d->map(options->offset, options->size, options->flags);
 return (returnValue->address != nullptr);
 }
 if (extension == UnMapExtension) {
- const UnMapExtensionOption *options = (const UnMapExtensionOption*)option;
+ const auto *options = static_cast<const UnMapExtensionOption *>(option);
 return d->unmap(options->address);
 }
 return false;
diff --git a/src/corelib/io/qresource.h b/src/corelib/io/qresource.h
index 8f17c1f075c..0361f435b3a 100644
--- a/src/corelib/io/qresource.h
+++ b/src/corelib/io/qresource.h
@@ -61,7 +61,7 @@ public:
 ZstdCompression
 };

- QResource(const QString &file=QString(), const QLocale &locale=QLocale());
+ QResource(const QString &file = QString(), const QLocale &locale = QLocale());
 ~QResource();

 void setFileName(const QString &file);
diff --git a/src/corelib/io/qsettings_mac.cpp b/src/corelib/io/qsettings_mac.cpp
index 70a2fe49e5a..60ef7cdb266 100644
--- a/src/corelib/io/qsettings_mac.cpp
+++ b/src/corelib/io/qsettings_mac.cpp
@@ -95,7 +95,7 @@ static QCFType<CFPropertyListRef> macValue(const QVariant &value);
 static CFArrayRef macList(const QList<QVariant> &list)
 {
 int n = list.size();
- QVarLengthArray<QCFType<CFPropertyListRef> > cfvalues(n);
+ QVarLengthArray<QCFType<CFPropertyListRef>> cfvalues(n);
 for (int i = 0; i < n; ++i)
 cfvalues[i] = macValue(list.at(i));
 return CFArrayCreate(kCFAllocatorDefault, reinterpret_cast<const void **>(cfvalues.data()),
diff --git a/src/corelib/io/qsettings_p.h b/src/corelib/io/qsettings_p.h
index bd8a40a3243..f26c121437f 100644
--- a/src/corelib/io/qsettings_p.h
+++ b/src/corelib/io/qsettings_p.h
@@ -94,7 +94,7 @@ class QSettingsKey : public QString
 {
 public:
 inline QSettingsKey(const QString &key, Qt::CaseSensitivity cs, int position = -1)
- : QString(key), theOriginalKey(key), theOriginalKeyPosition(position)
+ : QString(key), theOriginalKey(key), theOriginalKeyPosition(position)
 {
 if (cs == Qt::CaseInsensitive)
 QString::operator=(toLower());
diff --git a/src/corelib/io/qstorageinfo.cpp b/src/corelib/io/qstorageinfo.cpp
index e2c1f0232f9..9054d8e8944 100644
--- a/src/corelib/io/qstorageinfo.cpp
+++ b/src/corelib/io/qstorageinfo.cpp
@@ -456,12 +456,12 @@ QDebug operator<<(QDebug debug, const QStorageInfo &s)
 debug << (d->ready ? " [ready]" : " [not ready]");
 if (d->bytesTotal > 0) {
 debug << ", bytesTotal=" << d->bytesTotal << ", bytesFree=" << d->bytesFree
- << ", bytesAvailable=" << d->bytesAvailable;
+ << ", bytesAvailable=" << d->bytesAvailable;
 }
 } else {
 debug << "invalid";
 }
- debug<< ')';
+ debug << ')';
 return debug;
 }
 #endif // !QT_NO_DEBUG_STREAM
diff --git a/src/corelib/io/qurlquery.h b/src/corelib/io/qurlquery.h
index de474924cef..aba5df6a579 100644
--- a/src/corelib/io/qurlquery.h
+++ b/src/corelib/io/qurlquery.h
@@ -106,6 +106,7 @@ private:
 friend class QUrl;
 friend Q_CORE_EXPORT size_t qHash(const QUrlQuery &key, size_t seed) noexcept;
 QSharedDataPointer<QUrlQueryPrivate> d;
+
 public:
 typedef QSharedDataPointer<QUrlQueryPrivate> DataPtr;
 inline DataPtr &data_ptr() { return d; }