[qt/qt/qtbase]: Summary of bulk changes made
KDE Git Services - Bulk Change <[email protected]>
| Newsgroups | gmane.comp.kde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Git repository change summary for qt/qt/qtbase Pushed by mirror-service into branch 'dev'. Changed from 2abd933db0ffd7d60dab52cf6448b3631973510b to 761a7b10943f38f45d2c26590b7a1457ba918ca5 Acknowledgement was received that this change introduces only existing code that has been pushed to another public open source repository. This change contains the following new commits: Git commit 8f63540c38189ff0a94123b7d488c32b9642cad9 by Marc Mutz on 25/07/2026 at 18:33.. QTextStream: consolidate Private::status init'ing Coverity complained that the Private(QTextStream *) ctor didn't initialize the `status` field, but it missed that all callers of the constructor set it explicitly. Since it's also explicitly set after every Private::reset(), which the ctor also calls, it makes sense to move the status init to reset(), instead of the current shotgun-surgery style. Do that. Not picking back all the way, since it's a FP and there's a non-negligible chance that older branches didn't call reset() in the Private ctor (didn't check, just mentioning it as a risk). Pick-to: 6.12 Coverity-Id: 11685 Change-Id: I425b02f7db9a241c16c643bb9e63d166bac0d64a Reviewed-by: Thiago Macieira <[email protected]> https://invent.kde.org/qt/qt/qtbase/-/commit/8f63540c38189ff0a94123b7d488c32b9642cad9 Git commit 6b453e7e89f999cd487cd09ee90c0fa9f4af5f0a by Marc Mutz on 25/07/2026 at 18:33.. QAuthenticator: fix QBAV backed by destroyed QBA The old code iterated over the temporary result of QHttpHeaders::values(WellKnownHeader), and stored substrings in a QByteArrayView that outlives the for loop, therefore the temporary. Depending on how, exactly, QHttpHeaders returns these QByteArrays, the QByteArrayView might end up referencing data that was uniquely held by one of the QByteArrays in the list, and freed, after said byte array's destruction. The code also detaches the temporary QList, so kill two birds with one stone and store the result in a local const variable before the iteration, ensuring (const) no detaches and (local variable) that the byte arrays outlive the QByteArrayView referencing them. Amends 15b0bd69ff2a3ac967ddb98b5fc3c3ce8c3d5b4b (6.8). Detach found by Axivion / Clazy. Axivion-Id: qt_Essentials_qtnetwork_dev_linux:SV524 Pick-to: 6.12 6.11 6.8 Change-Id: Ic0bfe4165b9f45599a36e45123019386b2171853 Reviewed-by: Thiago Macieira <[email protected]> https://invent.kde.org/qt/qt/qtbase/-/commit/6b453e7e89f999cd487cd09ee90c0fa9f4af5f0a Git commit 61ab70658d01fe61db0c8a2467bb645616a6aaed by Marc Mutz on 25/07/2026 at 18:34.. QAuthenticator: add a missing move() The lambda takes the QString argument by value, presumably because it's only ever passed rvalues, and wants to benefit from C++17 GCE, but it didn't use std::move() when stowing the string away. Add the missing std::move(), fixing a clazy-function-args-by-ref running in my QtCreator. Amends 1cf84e7fd21a98a75b851b03467acf43c27d4504 (picked to 5.15). Pick-to: 6.12 6.11 6.8 Change-Id: I52e78b85bab940eafa06c002685889076178246a Reviewed-by: Thiago Macieira <[email protected]> https://invent.kde.org/qt/qt/qtbase/-/commit/61ab70658d01fe61db0c8a2467bb645616a6aaed Git commit 761a7b10943f38f45d2c26590b7a1457ba918ca5 by Marc Mutz on 25/07/2026 at 18:34.. QAuthenticator: enable -Wswitch in Private::parseHttpResponse() The old code disabled it due to the use of a default: case label. Remove it and list all enumerators instead. Amends the start of the public history. Pick-to: 6.12 6.11 6.8 Change-Id: I6460a80ec69d6793d107fb24f9c4c2ac8b0752d9 Reviewed-by: Thiago Macieira <[email protected]> https://invent.kde.org/qt/qt/qtbase/-/commit/761a7b10943f38f45d2c26590b7a1457ba918ca5