aboutsummaryrefslogtreecommitdiffstats
path: root/src/plugins/valgrind/xmlprotocol/parser.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Valgrind: Update xml protocol handlingChristian Stenger6 days1-2/+6
| | | | | | | | We now support protocol versions from 4 to 6. Fixes: QTCREATORBUG-33759 Change-Id: I5c2666be5913dcd5efa91ec158abc0fed2f00a37 Reviewed-by: hjk <hjk@qt.io>
* TaskTree: Switch to qt-ified QtTaskTreeJarek Kobus2025-10-311-3/+3
| | | | | | | Remove the old Tasking lib. Change-Id: Iacb4939cc873e25d6039efae7dd23510a7baa5f9 Reviewed-by: hjk <hjk@qt.io>
* Avoid QtConcurrent module importsChristian Stenger2025-08-261-0/+1
| | | | | | | | | ...as module imports pull in all headers of the module including all headers of modules they depend on. See discussion around QTBUG-88831. Change-Id: Idef4ca1b6c3c5ce9c8085d469bf3942ad0d9b4f9 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* Valgrind: Use TaskTree for ParserJarek Kobus2025-08-221-32/+33
| | | | | Change-Id: I2a134ffe5690aa85e3bc47f15d56fcdfed8f515b Reviewed-by: hjk <hjk@qt.io>
* Replace most expected_str by Utils::Resulthjk2025-04-111-1/+1
| | | | | | | | Keep it in Axivion dto, which is generated. Change-Id: I83a12de6234ac7b0218b369875bdc72d25dbadfb Reviewed-by: Eike Ziller <eike.ziller@qt.io> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* Utils: Replace Result class by type alias to std::expected<T, QString>hjk2025-04-111-4/+4
| | | | | | | | | | | | | | ... to be able to conveniently return also non-void cases without being exposed to the syntax of expected. The price for the more general approach is some uglification of the void case: The previous 'Result' is now equivalent to 'Result<>', which needs to be spelled out in function signatures, and some changes to the special cases. Change-Id: Ic5026e237ef2077a0765cdb8287122cae99d699f Reviewed-by: Marcus Tillmanns <marcus.tillmanns@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Valgrind: Replace Parser::done() args with Utils::ResultJarek Kobus2024-11-211-5/+5
| | | | | Change-Id: Iaed1f47d2dfc5195c83c3ef8c5141af1a1f17832 Reviewed-by: hjk <hjk@qt.io>
* Remove PluginManager::futureSynchronizer()Eike Ziller2024-05-211-3/+1
| | | | | | | Use the global synchronizer from Utils::futureSynchronizer() directly Change-Id: Ic8843bc1ff7951c041529a258f36117f08ec4b35 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* Merge remote-tracking branch 'origin/12.0'Eike Ziller2023-11-241-1/+1
|\ | | | | | | | | | | | | Conflicts: src/plugins/valgrind/xmlprotocol/parser.cpp Change-Id: I905a5c4f7334f733e99359fbfccd1572b4007696
| * Valgrind: Fix accessing empty optionalChristian Stenger2023-11-241-1/+1
| | | | | | | | | | Change-Id: I3d4d8ed9b919a6c0eb8d5530c9d8e2723a8cc1ae Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* | TaskTree: Unify TaskInterface::done(DoneResult) signalJarek Kobus2023-11-171-3/+4
|/ | | | | | | | | | | | Change the argument of TaskInterface::done() signal from bool into DoneResult. Make it consistent with other TaskTree API. Introduce toDoneResult(bool success) helper. Change-Id: I7b3041d7c1ed0317c76adbc1fd37448231e85f82 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: hjk <hjk@qt.io>
* Fix UI text capitalization, punctuation, and phrasingLeena Miettinen2023-10-201-4/+4
| | | | | | Change-Id: I41951660a464601063e5cedd417db2cb7cd37e44 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
* Valgrind: Make socket parentless when passed to ParserJarek Kobus2023-08-161-0/+1
| | | | | | | | | Otherwise, when the socket is a child of QTcpServer, it may be deleted together with a server, while it's still held in Parser instance. Change-Id: I600e9cef1268ffc93e1e86d317e087df8b4d70cb Reviewed-by: hjk <hjk@qt.io>
* Valgrind: Refactor ParserJarek Kobus2023-08-141-218/+345
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Make it implicitly threaded. Get rid of ThreadedParser. Don't move QTcpSocket instances, received by QTcpServer::nextPendingConnection(), into another thread, as docs says it's not safe. Use wait condition for waiting for new data. Don't potentially leak the Parser in loadXmlLogFile(): store a unique pointer to the running parser. Introduce Parser::setData() method and use it with QFile device in loadXmlLogFile(). Pass QAbstractSocket into the parser instead of QIODevice and get rid of downcasting. The QIODevice couldn't really work with non-socket and non-file devices (like e.g. QProcess), as it lacks general error and finished reporting signals (it has only general readyRead() signal). Change-Id: I9352aec694564d4d2a26898841ed964bed470d82 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* Valgrind: Make Parser's API similar to the ThreadedParser's APIJarek Kobus2023-08-091-12/+16
| | | | | | Change-Id: I39634c881dac44fc857a4aa2049e9c2618daec9b Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* Valgrind: Merge parsers' finished() and internalError() signalsJarek Kobus2023-08-081-9/+7
| | | | | | | Replace them with done() signal. Change-Id: I5f03df927ddefd803d9cb0b06d3f4c1601a15381 Reviewed-by: hjk <hjk@qt.io>
* Valgrind: Make tool name hash staticJarek Kobus2023-08-081-7/+13
| | | | | | | Change-Id: I326c45e1be293dc5066271839b26b7f22d53eabb Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: hjk <hjk@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* ValgrindPlugin: Use QList instead of QVectorJarek Kobus2023-08-081-8/+8
| | | | | | | Change-Id: Ib1bd223b73d1f7399008f91a3c26ff515ab03a0c Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: hjk <hjk@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* Valgrind: Use QMetaEnum for parsing enum valuesJarek Kobus2023-07-251-70/+12
| | | | | Change-Id: I43685e3fde662a57b6966d5f5f29d4138158b4d8 Reviewed-by: hjk <hjk@qt.io>
* Valgrind Parser: Hide Tool enum in cppJarek Kobus2023-07-251-16/+21
| | | | | | | Replace some explicit iterators with auto. Change-Id: I30aad955e02cbffbfd4ffe100d381e32202fea05 Reviewed-by: hjk <hjk@qt.io>
* Remove GPL-3.0+ from license identifiersKai Köhne2023-01-061-1/+1
| | | | | | | | | | | | | | | Since we also license under GPL-3.0 WITH Qt-GPL-exception-1.0, this applies only to a hypothetical newer version of GPL, that doesn't exist yet. If such a version emerges, we can still decide to relicense... While at it, replace (deprecated) GPL-3.0 with more explicit GPL-3.0-only Change was done by running find . -type f -exec perl -pi -e "s/LicenseRef-Qt-Commercial OR GPL-3.0\+ OR GPL-3.0 WITH Qt-GPL-exception-1.0/LicenseRef-Qt-Commercial OR GPL-3.0-only WITH Qt-GPL-exception-1.0/g" {} \; Change-Id: I5097e6ce8d10233993ee30d7e25120e2659eb10b Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Use SPDX license identifiersLucie Gérard2022-08-261-25/+2
| | | | | | | | | Replace the current license disclaimer in files by a SPDX-License-Identifier. Task-number: QTBUG-67283 Change-Id: I708fd1f9f2b73d60f57cc3568646929117825813 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Valgrind: Use Tr::trhjk2022-07-111-29/+16
| | | | | Change-Id: I77b27462d88079699e620ef459f951846c15c3e2 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Remove unused includes of QThread from cpp filesJarek Kobus2021-09-131-1/+0
| | | | | | Change-Id: I61b8e9182bf03fa96d2aabe399a6c93964fc8d03 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: hjk <hjk@qt.io>
* Qt6: Sprinkle QLatin1Strings for QStringView comparisonsEike Ziller2020-10-081-53/+51
| | | | | | Task-number: QTCREATORBUG-24098 Change-Id: I2fb79bcfd0537a6704a1ee8db840401d6beca7a0 Reviewed-by: hjk <hjk@qt.io>
* Fix build issues with Qt6Eike Ziller2020-09-181-11/+11
| | | | | | | | Change from QStringRef to QStringView at various places. Task-number: QTCREATORBUG-24098 Change-Id: Ia7a634fa26464fbb2962724d5f0e188cecc68801 Reviewed-by: hjk <hjk@qt.io>
* Valgrind: ModernizeAlessandro Portale2018-12-101-9/+7
| | | | | | | modernize-* Change-Id: I6db60dce78cf2575e36caa597b1f095adba34fd9 Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Valgrind: Strip QLatin1*Orgad Shaneh2018-12-031-70/+70
| | | | | Change-Id: If93ca890ab6d023ab786a5153f50a1dfa03764de Reviewed-by: hjk <hjk@qt.io>
* Valgrind: Fix crash when missing stacksHannes Domani2017-12-141-0/+4
| | | | | Change-Id: I69896f0d0c428f158bdb071b08a1c4b9f031183f Reviewed-by: hjk <hjk@qt.io>
* Update License according to agreement with Free Qt FoundationTobias Hunger2016-01-191-17/+12
| | | | | | | * Update files in src/plugins Change-Id: Ia5d77fad7d19d4bb3498e78661982f68729adb22 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* Valgrind: Introduce Frame::filePath().Christian Kandeler2015-06-261-1/+1
| | | | | Change-Id: I40d1b7f739ea905bbcca1a388bada3e077b7c137 Reviewed-by: hjk <hjk@theqtcompany.com>
* Don't mix iterator and const_iteratorKai Koehne2015-04-021-4/+4
| | | | | | | | | | | This avoids unnecessary detaches of the Qt container data. The mismatches where detected by defining QT_STRICT_ITERATORS; however, this define violates the ODR (causing linker errors), and therefore is not added permanently. Change-Id: Idd336a9c8b394214a820437ef1b92d2101f6101c GPush-Base: 62b0848b9cf357bcded4184c71d490bae56c773b Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
* Update LicenseEike Ziller2015-01-161-6/+6
| | | | | Change-Id: I711d5fb475ef814a1dc9d2822740e827f3f67125 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* License updateEike Ziller2014-10-091-7/+8
| | | | | Change-Id: I3c22ef2685d7aa589f5d0ab74d693653a4c32082 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* Incremented year in copyright infoRobert Loehning2014-01-081-1/+1
| | | | | | Change-Id: Ib5423fdd064e4546f848c0b640b0ed0514c26d3a Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com> Reviewed-by: Kai Koehne <kai.koehne@digia.com>
* Fix coding style for else statementsOrgad Shaneh2013-07-171-14/+11
| | | | | Change-Id: I1309db70e98d678e150388c76ce665e988fdf081 Reviewed-by: Eike Ziller <eike.ziller@digia.com>
* Merge remote-tracking branch 'origin/2.6'Oswald Buddenhagen2013-01-311-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/plugins/autotoolsprojectmanager/AutotoolsProjectManager.pluginspec.in src/plugins/debugger/qtmessageloghandler.cpp src/plugins/debugger/qtmessagelogwindow.cpp src/plugins/madde/maemodeployconfigurationwidget.cpp src/plugins/qmldesigner/components/integration/designdocumentcontroller.cpp src/plugins/qmldesigner/designercore/include/widgetqueryview.h src/plugins/qmldesigner/designercore/metainfo/metainfoparser.cpp src/plugins/qmldesigner/designercore/model/modelnodecontextmenu.cpp src/plugins/qmldesigner/designercore/model/modelnodecontextmenu.h src/plugins/qmlprojectmanager/qmlprojectapplicationwizard.cpp src/plugins/qnx/bardescriptormagicmatcher.h src/plugins/qt4projectmanager/profilekeywords.cpp src/plugins/remotelinux/deployablefilesperprofile.cpp src/plugins/remotelinux/deployablefilesperprofile.h src/plugins/remotelinux/deploymentinfo.cpp src/plugins/remotelinux/deploymentsettingsassistant.cpp src/plugins/remotelinux/profilesupdatedialog.cpp tests/auto/icheckbuild/ichecklib.cpp tests/auto/icheckbuild/parsemanager.cpp tests/auto/icheckbuild/parsemanager.h Change-Id: Ie465a578446a089e1c502d1cb1096e84ca058104
| * Incremented year in copyright infov2.6.2Robert Loehning2013-01-291-1/+1
| | | | | | | | | | Change-Id: Ic6a9ff0359625021ebc061d22db6811814534205 Reviewed-by: Kai Koehne <kai.koehne@digia.com>
* | Remove braces for single lines of conditionsOrgad Shaneh2013-01-081-2/+1
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | #!/usr/bin/env ruby Dir.glob('**/*.cpp') { |file| # skip ast (excluding paste, astpath, and canv'ast'imer) next if file =~ /ast[^eip]|keywords\.|qualifiers|preprocessor|names.cpp/i s = File.read(file) next if s.include?('qlalr') orig = s.dup s.gsub!(/\n *if [^\n]*{\n[^\n]*\n\s+}(\s+else if [^\n]* {\n[^\n]*\n\s+})*(\s+else {\n[^\n]*\n\s+})?\n/m) { |m| res = $& if res =~ /^\s*(\/\/|[A-Z_]{3,})/ # C++ comment or macro (Q_UNUSED, SDEBUG), do not touch braces res else res.gsub!('} else', 'else') res.gsub!(/\n +} *\n/m, "\n") res.gsub(/ *{$/, '') end } s.gsub!(/ *$/, '') File.open(file, 'wb').write(s) if s != orig } Change-Id: I3b30ee60df0986f66c02132c65fc38a3fbb6bbdc Reviewed-by: hjk <qthjk@ovi.com>
* Adjust license headershjk2012-10-051-22/+20
| | | | | Change-Id: Ice592c6de9951ee3b2c4cb52ed0bb3b6770e0825 Reviewed-by: Eike Ziller <eike.ziller@digia.com>
* Contact -> qt-project.orgEike Ziller2012-07-191-3/+1
| | | | | Change-Id: I7134d7de30bcf9f9dcfad42520dd45ee083a852d Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* Removed module names from #include directives.Erik Verbruggen2012-02-151-7/+7
| | | | | | | | Getting the #include directives ready for Qt5. This includes the new-project wizards. Change-Id: Ia9261f1e8faec06b9285b694d2b7e9a095978d2b Reviewed-by: Eike Ziller <eike.ziller@nokia.com>
* Long live the king!hjk2012-01-261-1/+1
| | | | | Change-Id: I2b72b34c0cfeafc8bdbaf49b83ff723544f2b6e2 Reviewed-by: Daniel Teske <daniel.teske@nokia.com>
* all: s/info@qt.nokia.com/qt-info@nokia.com/hjk2011-11-031-2/+2
| | | | | Change-Id: If18afb5d4665924e7d9250dccbc60a65e6daa75e Reviewed-by: Eike Ziller <eike.ziller@nokia.com>
* analyzer: code cosmeticshjk2011-08-171-75/+89
| | | | | | Change-Id: Id00cf1be9d07e47f7b61b8cc7e940629c84a48c1 Reviewed-on: http://codereview.qt.nokia.com/2707 Reviewed-by: hjk <qthjk@ovi.com>
* valgrind: code cosmeticshjk2011-07-131-0/+753
Change-Id: I695c7d40aeb0b0b1e6f8cc0856e5fa5d9d1ce2d1 Reviewed-on: http://codereview.qt.nokia.com/1600 Reviewed-by: hjk <qthjk@ovi.com>
generated by cgit v1.2.3 (git 2.25.1) at 2025-12-18 08:36:34 +0000