aboutsummaryrefslogtreecommitdiffstats
path: root/src/plugins/projectexplorer/outputparser_test.cpp
Commit message (Collapse)AuthorAgeFilesLines
* ProjectExplorer: Delay-create TaskHubhjk2024-02-021-2/+2
| | | | | | | Effectively only by a few lines. Change-Id: Ieb548d0a2f5ced033881106c76adc9a5e3e16663 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* ProjectExplorer: Move outputparser test class creation classhjk2024-01-181-21/+44
| | | | | | | ... closer to the tested code. Change-Id: Icdc2efcae8bfa6129ed641116ca6e8c917d8c0f3 Reviewed-by: Christian Kandeler <christian.kandeler@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-24/+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>
* ProjectExplorer: Add output parser for AddressSanitizer messagesChristian Kandeler2022-04-131-0/+2
| | | | | | Change-Id: I9107a4f23998ed95f374c7d61c9ee1e6e6e2d437 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* Task: Add column information to compiler output tasksAndre Hartmann2021-05-101-0/+1
| | | | | | | | | | | | | | | | | | | | Allows to directly jump to the problematic code location in the editor. Done for GCC and Clang which both use the format: file:line:column Not done for MSVC, as this compiler uses the format: file(line) and does not include column information. It seems like clang-cl does use the format: file(line,column) but the existing parsers were not prepared for that and were therefore left unchanged for now. Change-Id: I182634e1c5c941b19801ecafb69fa2b8f91f9d7a Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* PE: Fix expectations of LinkSpecs test on WindowsChristian Stenger2021-03-111-0/+3
| | | | | | | | | | | | The output parser detects Windows paths on Unix differently which makes the format differ from the one that is generated on a Windows host. Beside this remove BOM from msvcparser again and do not perform the format check if there is no expected format specified. Amends 9e4e9924ad26c708. Change-Id: Icb0d292a8908cdd2f86b13fde6d2d287f3a96dd6 Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Unit-test LinkSpecs of tasksBernhard Beschow2021-03-091-0/+6
| | | | | Change-Id: Idc86c8fcb9a915490e4a67beb7a057672096861f Reviewed-by: hjk <hjk@qt.io>
* Don't detach temporary containersAlessandro Portale2021-02-241-2/+2
| | | | | Change-Id: I736cfc61c81995a81b056b3e3e5703ded5912525 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* ProjectExplorer: Fix setting up test terminatorChristian Stenger2020-09-281-1/+6
| | | | | | | | | | When setting up the test terminator the used line parser has been setup already. Ensure the test terminator has the same search directories set up to avoid an assert. Change-Id: Ic881533f7735f769dd49d22860141c9213e77e09 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Merge remote-tracking branch 'origin/4.13' into masterEike Ziller2020-09-071-3/+2
|\ | | | | | | Change-Id: Iab93dc48b4a38fb4c698481bbf02a5a65e08206b
| * OutputFormatter: Fix visual glitchqds/v4.3.2qds/v1.6.0-rc1qds/1.6.0-rc1Christian Kandeler2020-09-071-3/+2
| | | | | | | | | | | | | | | | | | | | | | When inserting a line into an output window, we have to delay appending the line feed character. Otherwise strange visual effects appear under certain circumstances. I have no idea why. Fixes: QTCREATORBUG-24411 Change-Id: If8842ae4d9db36d514996b1f34dcca0432fafbfc Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* | GCC parser: Recognize messages referring to scopeChristian Kandeler2020-07-281-0/+4
|/ | | | | | | | | | ... such as "foo.cpp: In member function 'bar': ...". These weren't caught at all, because the file name has no line number. They did show up in the issues pane, because the ld parser accidentally caught them, but they were not properly merged with their context. Change-Id: I29a757879c21a051cb31c65497b3ad2bfff2315e Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Give the Task class an explicit summaryChristian Kandeler2020-05-141-1/+1
| | | | | | | | | | | | | We'd like to create more useful tasks from compiler output, that is, try harder to identify consecutive lines that refer to the same issue and create one task for them, rather than one for each line. In such "aggregate" tasks, the first line will not necessarily carry the main information. Therefore, we make it explicit what this main information is by introducing a dedicated summary member. Also streamline the font handling for compile tasks. Change-Id: I933f2643a13c710dab1ab548c56669b129026eb5 Reviewed-by: hjk <hjk@qt.io>
* Merge output formatters and output parsersChristian Kandeler2020-04-231-4/+69
| | | | | | | | | | | | | | | | | | | | | | | Now only one piece of code needs to be written to both linkify output in an output pane and create tasks for it in the issues pane. The calling sites are also simplified. For instance, until now, build steps had to feed their output parsers manually and then push the created tasks up the signal stack in parallel with the actual output, which the build manager relied upon for cross-linking the output pane content. Afterwards, the output would get forwarded to the formatter (and parsed for ANSI escape codes a second time). In contrast, a build step now just forwards the process output, and task parsing as well as output formatting is done centrally further up the stack. Concrete user-visible improvements so far: - File paths in compiler/linker messages are clickable links now. - QtTest applications now create clickable links also when run as part of a build step, not just in the app output pane. Task-number: QTCREATORBUG-22665 Change-Id: Ic9fb95b2d97f2520ab3ec653315e9219466ec08d Reviewed-by: Christian Kandeler <christian.kandeler@qt.io> Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* ProjectExplorer: Split up the IOutputParser classChristian Kandeler2020-04-161-1/+1
| | | | | | | | For symmetry with Utils::OutputFormatter. Task-number: QTCREATORBUG-22665 Change-Id: I148fed69dba042ad3ef26e080829c31cd3f357fd Reviewed-by: hjk <hjk@qt.io>
* Output parsers: Replace the chaining approachChristian Kandeler2020-04-091-8/+8
| | | | | | | | | | | Use "flat" aggregation instead. This is another step towards the formatter/parser merger. Along the way, also fix some some subclasses (mostly in BareMetal) that erroneously forwarded handled output to other parsers. Task-number: QTCREATORBUG-22665 Change-Id: I12947349ca663d2e6bbfc99efd069d69e2b54969 Reviewed-by: hjk <hjk@qt.io>
* IOutputParser: Replace std{Out,Err} with a single parameterized functionChristian Kandeler2020-04-081-8/+5
| | | | | | | | Another step towards parser/formatter unification. Task-number: QTCREATORBUG-22665 Change-Id: I6de86b3aee2c54585cdd4b06d21b0ea67300aeac Reviewed-by: hjk <hjk@qt.io>
* Output parsers: Generalize the search directory conceptChristian Kandeler2020-04-071-27/+7
| | | | | | | | | | All parsers can now have search directories, not just the GnuMakeParser. This allows us to get rid of the "task mangling", removing another instance where the order of parsers in the chain mattered. Task-number: QTCREATORBUG-22665 Change-Id: Id0d55522ae6800afd9f50ff36546224b0d8bb382 Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Let IOutputParser handle newlinesChristian Kandeler2020-04-071-8/+5
| | | | | | | | | | This makes IOutputParser structurally more similar to Utils::OutputFormatter, which makes it simpler to explore possibilities of somehow uniting these two related classes. Task-number: QTCREATORBUG-22665 Change-Id: Ibb12ab6c8c785d863b9a921102a929864d0a5251 Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Clean up IOutputParser interfaceChristian Kandeler2020-04-031-41/+5
| | | | | | | | | | | | - Remove unneeded/unused functions. - De-virtualize where possible. In particular, after untangling a number of self-referential redirections, it became apparent that the outputAdded() infrastructure was entirely unused. Change-Id: I51e1beed008df2727b42494b087efa476342397e Reviewed-by: hjk <hjk@qt.io>
* Avoid warning on empty expressionshjk2019-07-231-3/+3
| | | | | | | | For some reason, Q_UNUSED includes already a semicolon, adding one on the user side creates an additional empty statement. Change-Id: I9c5e8fac381345a60792cb75e2938fd53958d3b0 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Utils: Rename FileName to FilePathhjk2019-05-281-1/+1
| | | | | | | | More in line with QFileInfo terminonlogy which appears to be best-of-breed within Qt. Change-Id: I1d051ff1c8363ebd4ee56376451df45216c4c9ab Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* ProjectExplorer: Introduce a alias for QList<Tasks>hjk2019-05-281-1/+1
| | | | | Change-Id: I91391ad22b420926b0f512cac23cfe009048b218 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* ProjectExplorer: ModernizeAlessandro Portale2018-07-161-2/+1
| | | | | | | | | | | | | | modernize-use-auto modernize-use-nullptr modernize-use-override modernize-use-using modernize-use-default-member-init modernize-use-equals-default modernize-use-transparent-functors Change-Id: Iebed22caa2e733d292f334e956e3d16b844e14e3 Reviewed-by: Orgad Shaneh <orgads@gmail.com> Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* OutputParserTester: ModernizeAndre Hartmann2018-06-151-14/+12
| | | | | Change-Id: Ifa8cd265d32f7fb9f875ede2b05ffb3616fcca62 Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Give the values of BuildStep::OutputFormat better namesChristian Kandeler2017-01-191-1/+1
| | | | | | | | The old ones did not convey their meaning very well. In particular, NormalOutput and MessageOutput were easily confused. Change-Id: Ia0a8c1b1c366ab3f5c59f751b37b8b1f68f6831d Reviewed-by: Tobias Hunger <tobias.hunger@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>
* Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-05-261-2/+11
|\ | | | | | | Change-Id: I35ba4cc7f7052699c3006545514c866be3cb5fdd
| * Adapt MSVC output parser to MSVC 2015.Friedemann Kleint2015-05-221-2/+11
| | | | | | | | | | | | | | | | The space character after the line number was removed from the compiler messages. Change-Id: Ia249a6d4416744c67aada2eb2c5c26cf87bd7ef7 Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
* | Tasks: Make the linking of compile output to Tasks more robustDaniel Teske2015-04-211-1/+3
|/ | | | | | | | | | | | | | | | | | | | Clicking on error messages is supposed to jump to the editor. And "Show Output" on the task is supposed to select the error in the output. The old code just registered the task for the last line of output. This broke for every parser that allowed for error messages that spanned multiple lines. And was obviously also incorrect for tasks that weren't generated due to compile output. Fix both of those issues by giving the IOutputParsers more control on which lines are linked to a task. Task-number: QTCREATORBUG-14136 Change-Id: I095922c9875620dabfb7d406f6b152c8a9b25b62 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com> 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>
* Merge "Merge remote-tracking branch 'origin/3.2'"Eike Ziller2014-10-141-7/+8
|\
| * License updateEike Ziller2014-10-091-7/+8
| | | | | | | | | | Change-Id: I3c22ef2685d7aa589f5d0ab74d693653a4c32082 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | ProjectExplorer: Remove redundant namespace qualificationsOrgad Shaneh2014-10-141-2/+2
|/ | | | | Change-Id: Id9034e31f3f61c38894111951534dd0d59b746dc Reviewed-by: hjk <hjk121@nokiamail.com>
* ProjectExplorer: Pass some values by referenceOrgad Shaneh2014-05-201-2/+2
| | | | | Change-Id: I628c317fc86f1f4b33ca453018d54fb86babc1e8 Reviewed-by: Eike Ziller <eike.ziller@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>
* Rename "[Mm]ethod(s)" to "[Ff]unction(s)"Nikolai Kosjar2013-10-101-1/+1
| | | | | | | | | | | Only methods as programming functions are affected. Besides renaming some actions like "Switch Between Function Declaration/Definition" this mostly touches (api) code comments. This is a follow-up patch to commit 872bfb7. Change-Id: Icb65e8d73b59a022f8885b14df497169543a3b92 Reviewed-by: hjk <hjk121@nokiamail.com>
* OutputParser: Allow to flush pending tasks.Tobias Hunger2013-05-061-0/+1
| | | | | | | | | | | | | | | Add a way to flush out tasks from the outputparsers. This is necessary to make parsers that keep state more robust. Flush parsers (once) before adding any new task. This keeps the sequence intact. Flush all parsers once the parsing is done to make sure there is no task queued somewhere. Task-number: QTCREATORBUG-9195 Change-Id: Icd37df1f470cb73123ad286d6900ad1047a1d512 Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
* Make test bench for IOutputParser tests more stableTobias Hunger2013-05-061-24/+23
| | | | | Change-Id: I39a08912efdcf34ef680d508c41c4df9034125dc Reviewed-by: Daniel Teske <daniel.teske@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>
* | Add whitespace after control keywordsOrgad Shaneh2012-11-281-1/+1
|/ | | | | | | | find -name \*.cpp -o -name \*.h | \ xargs sed -Ei 's/ (for|foreach|if|switch|while)\(/ \1 (/g' Change-Id: I9efdff4bf0c8c01a52baaaeb75198483c77b0390 Reviewed-by: hjk <qthjk@ovi.com>
* Adjust license headershjk2012-10-051-21/+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>
* Long live the king!hjk2012-01-261-1/+1
| | | | | Change-Id: I2b72b34c0cfeafc8bdbaf49b83ff723544f2b6e2 Reviewed-by: Daniel Teske <daniel.teske@nokia.com>
* ProjectExplorer: Compile with QT_NO_CAST_FROM_ASCII.Friedemann Kleint2012-01-091-2/+2
| | | | | | | | - Add missing translations - Remove some unneeded conversions. Change-Id: Ia30e5c838099e52a9f38ca4854395c10c0391075 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>
* Header cleanup in ProjectExplorer.Friedemann Kleint2011-08-181-0/+1
| | | | | | Change-Id: I19f100003427ba43aaae5ba0dc7078088017cb09 Reviewed-on: http://codereview.qt.nokia.com/3178 Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
* outputparser: Fix overriding a virtual methodTobias Hunger2011-06-011-2/+2
| | | | | | | Change-Id: I2e0f2525e674e1b1d6e337d211adb16586478518 Reviewed-on: http://codereview.qt.nokia.com/300 Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com> Reviewed-by: Tobias Hunger <tobias.hunger@nokia.com>
generated by cgit v1.2.3 (git 2.25.1) at 2024-05-10 06:45:58 +0000