summaryrefslogtreecommitdiff
path: root/src/plugins/valgrind/memcheckengine.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Valgrind: Merge memcheck{engine,tool}.* fileshjk2017-09-251-185/+0
| | | | | | | | | This is purely mechanical in preparation of moving some per-run items from the MemCheckTool singleton to MemCheckToolRunner. Change-Id: I0fcaf6e90b2d63ca8f3c3eb7130ed73ed494f35b Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Merge DebuggerStartParameter into *RunParametershjk2017-09-131-1/+2
| | | | | | | | | | The users (typically target specific DebuggerRunTool derived classes) are meant to use the individual setter functions nowadays, not the set up the full structure, so the members are a true implementation detail now. Change-Id: Ida04801e3230a2fe8bbadde8845e58c3077c87a5 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* iOS: Adapt to new setter-based DebuggerRunTool interfacehjk2017-09-041-1/+1
| | | | | Change-Id: I96f113b831dcbd3735f743c8f12042e9b17903a4 Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io>
* Valgrind: Extend and use new DebuggerRunTool interfacehjk2017-08-241-12/+10
| | | | | Change-Id: I5e015a484e67175be12d538e7f814d61001ad7bb Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Rename worker dependencies to "startDependencies"Ulf Hermann2017-08-081-1/+1
| | | | | | | | There will be stopDependencies, too, which apply when stopping the runcontrol. Change-Id: Id72771d28cbb6b254572c9f93db93e0d054b890f Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Rename RunControl::finished to stoppedhjk2017-07-121-1/+1
| | | | | | | That's what it is. Change-Id: I8cf9af23bb7cafe1cde8f165fdbf85cdeb79f7e4 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Valgrind: Keep tool names in the *ToolRunners onlyhjk2017-06-301-2/+1
| | | | | | | Slightly less convoluted. Change-Id: I07410b49db05ea862fb82f3bffc25a5d7fb5a104 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Valgrind: Merge ValgrindRunner and ValgrindProcesshjk2017-06-301-1/+0
| | | | | | | There was a 1:1 matching remaining. Change-Id: I619bedcda867b642eab37396a0bd48bcb3a5829a Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Valgrind: Remove CallgrindController's use of ValgrindProcesshjk2017-06-291-4/+2
| | | | | | | | | | It's simply one-shot command execution, using (only some part of) the ValgrindProcess machinery that just happens to also to wrap a process is conceptually different from ValgrindProcess that "is" the valgrind-with-debuggee entity (and an unneeded dependency) Change-Id: I57a2c3d1cab6b15e59cb41b8e131948c170297b6 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Valgrind: Fix Memcheck startup on remote linuxhjk2017-06-281-12/+36
| | | | | | | Remote CallGrind is still dysfunctional. Change-Id: Ib9ab537dc068c94c7e61ac48b1a4b9d655ccb60f Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Valgrind: Add a ValgrindRunner member to the ValgrindToolRunner basehjk2017-06-261-10/+5
| | | | | | | ... instead of having one in each derived class. Change-Id: Icd121ce46b1d161bd2d59eaeaad8363528dc3c23 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Valgrind: Make the (threaded) parser a proper member of the runnerhjk2017-06-231-6/+4
| | | | | | | | Simplifies user code, and it was only ever used in a 1:1 relation, even in the tests. Change-Id: I3ce4fc83a361aceb730c05420efdb4ea52d37cda Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Valgrind: Merge MemCheckRunner and ValgrindRunner basehjk2017-06-231-2/+2
| | | | | | | | | | It's mostly the xml/log handling, which can be re-used in other tools. The change is purely mechanical, including some style fixes and some modernization. Change-Id: I6b44ae71451e8d3e26df40b9af9b4ec70038a92d Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Valgrind: Merge MemCheck{,WithGdb}ToolRunnerhjk2017-06-221-33/+28
| | | | | Change-Id: Iaf0707cf5c06ef29c33da835133948256869c76a Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Valgrind: Announce 'started', not 'stopped' on startuphjk2017-05-161-0/+1
| | | | | | | Also provide a name for the workers. Change-Id: I323c498db1eeb8225170aae080fdf084aa53e14a Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Valgrind: Move to new target/tool split for local setupshjk2017-05-021-24/+26
| | | | | Change-Id: I1167fdc147600c36149c13731d0680b858acf4fb Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Remove DebuggerRunControlhjk2017-04-281-2/+2
| | | | | | | Use plain RunControl + DebuggerRunTool combo instead. Change-Id: Ib71b5eab50da667b9d71dcc6689d2643ad8ecdee Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* ProjectExplorer: Run RunControl::{start,stop} always asynchronouslyhjk2017-03-241-3/+3
| | | | | | | | | | | | | | | | This introduces a mini-state-"machine" to handle RunControl states Intialized->[Starting->Running->Stopping->Stopped->]*->Finished. Needing time between trying to start and getting feedback is nowadays the normal setup for all remote targets as well as for most local tools. Making that the default for all runs simplifies the code and provides an opportunity to (a) fix some currently wrong reports of "stopped immediately" and (b) to remove target-specific (WinRT) or tool-specific (Valgrind, GammaRay) state members doing essentially the same. Change-Id: I7f52fee41144188ee8389e922fdc265f8c0a6459 Reviewed-by: David Schulz <david.schulz@qt.io> Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* Valgrind: Split MemCheck and CallGrind factorieshjk2016-05-101-2/+2
| | | | | | | | The tools are separated everywhere, lumping them into the same RunControlFactory removes modularity artificially. Change-Id: I8d9e917bb114a1898a0c293f18d3bf78a52075aa Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Merge debug mode and analyze modehjk2016-03-071-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | On the user-visible side, only the 'Analyze' mode button disappears, and instead a combobox to switch between different tools in appears in the Debug mode toolbar. Internally, that's quite some re-organzition: The centralized 'Analyze mode is busy' flag is gone, allowing us to run e.g. ClangStaticAnalyzer and MemCheck in parallel. Analyzer tools and debugger now share the same mechanism to generate/load/save dock widgets. Analyzer tools now create and handle their own start/stop button when appropriate. In general, Analyzer tools can create/handle more than one run control at a time. Further consolidation is possible, e.g. RunControl state handling could be merged into the base ProjectExplorer::RunControl to avoid the still existing duplication in ~15 instances. Change-Id: I91e5940ebc4211f98056d507cf2f7b5f8efe7f07 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Remove AnalyzerRunControl::runnable() and .connection()hjk2016-01-291-1/+1
| | | | | | | | These re-implementations are conceptually "too specific". Rather let the indiviual tools handle there expectations themselves. Change-Id: I0bbea407b2241816a40d19eb1dbb0a7589cbda7b Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Use StandardRunnable in DebuggerStartParametershjk2016-01-281-1/+1
| | | | | | | | This is a mechanical replacement for the former executable, processArgs, inferiorEnvironment and workingDirectory members. Change-Id: I4160e01427ed801df9b729f1f31d0a2ca48159b5 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Use a StandardRunnable instead of an AnalyzerRunnablehjk2016-01-271-1/+1
| | | | | | | ... with a lot potential to code consolidation. Change-Id: I4d3a7fcc1cc6ae8763799f18cf9701695f387791 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Remove AnalyzerStartParameters from AnalyzerRunControl ctorhjk2016-01-201-6/+4
| | | | | Change-Id: I0f7e982a9144469869e0c42daedfb411789511ca Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* Analyzer: Split AnalyzerStartParametershjk2016-01-201-2/+1
| | | | | | | | | | | ... into AnalyzerRunnable and AnalyzerConnection and use the RunControl's storage instead of an own copy in AnalyzerRunControl. This is an intermediate step on the way to remove AnalyzerStartParameters. Change-Id: Iee7c38781b2fd6ef030dcdada1993684cbb34c74 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* 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>
* Analyzer: Slim down AnalyzerStartParametershjk2016-01-071-3/+4
| | | | | | | | | | | | | | | * SysRoot can always be determined from kit. * Pass around RunMode as extra parameter not as part of AnalyzerStartParameters. That's closer to the pattern used elsewhere. * Environment was always initialized from the runconfig's EnvironmentAspect. The tools can do that directly. * Provide setter for display name for cases where it is not equal to RunConfiguration::displayName Change-Id: I811a0d7cdeb55cc37a16a593b3942abb567a2150 Reviewed-by: BogDan Vatra <bogdan@kdab.com> Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com>
* Debugger: Continue DebuggerStartParameter cleanuphjk2015-06-261-25/+3
| | | | | | | | | | | | | - Move sysRoot, debuggerCommand, targetAbi to DebuggerRunParameters, they are always the one coming from the kit. - Move projectSource{Directory,Files} to DebuggerRunParameters, they are alway coming from the runConfiguration's project - Pass RunConfiguration always as separate parameter, that's what related code does. Change-Id: I9965a052237af53fa2d170701bc88b552cab12ed Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com> Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Use free Debugger::createDebuggerRunControl() functionhjk2015-05-291-1/+1
| | | | | | | | | | | | Allows us to un-export DebuggerRunControlFactory in an attempt to slim down the publicly visible Debugger plugin interface Change-Id: I0c2e7351ffa7a9d16586172816635be8ff40172e Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com> Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com> Reviewed-by: BogDan Vatra <bogdan@kde.org> Reviewed-by: David Schulz <david.schulz@theqtcompany.com> Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
* Clear old analyzer issues on tool start.Christian Kandeler2015-03-121-4/+0
| | | | | Change-Id: I54f7791c15e05ca5c85c8b4059f80b050de57282 Reviewed-by: hjk <hjk@theqtcompany.com>
* Merge commit '3c85058694ee2e41658d17f524fb48f0b187d2fe'Eike Ziller2015-02-121-6/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/libs/utils/tooltip/tipcontents.cpp src/libs/utils/tooltip/tipcontents.h src/plugins/android/androiddeployqtstep.cpp src/plugins/baremetal/baremetalconstants.h src/plugins/baremetal/baremetaldevice.cpp src/plugins/baremetal/baremetaldevice.h src/plugins/baremetal/baremetaldeviceconfigurationwidget.cpp src/plugins/baremetal/baremetaldeviceconfigurationwidget.h src/plugins/baremetal/baremetaldeviceconfigurationwizard.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.h src/plugins/baremetal/baremetalplugin.cpp src/plugins/baremetal/baremetalplugin.h src/plugins/baremetal/baremetalruncontrolfactory.cpp src/plugins/baremetal/baremetalruncontrolfactory.h src/plugins/cppeditor/cppcodemodelinspectordialog.cpp src/plugins/cppeditor/cppdoxygen_test.cpp src/plugins/cppeditor/cppdoxygen_test.h src/plugins/debugger/breakpointmarker.cpp src/plugins/debugger/debuggeritemmodel.cpp src/plugins/debugger/debuggeritemmodel.h src/plugins/debugger/loadcoredialog.cpp src/plugins/genericprojectmanager/cppmodelmanagerhelper.cpp src/plugins/projectexplorer/addnewmodel.cpp src/plugins/projectexplorer/addnewmodel.h src/plugins/projectexplorer/jsonwizard/jsonfieldpage.cpp src/plugins/qmlprofiler/abstracttimelinemodel.cpp src/plugins/qmlprofiler/abstracttimelinemodel.h src/plugins/qmlprofiler/notesmodel.cpp src/plugins/qmlprofiler/qml/CategoryLabel.qml src/plugins/qmlprofiler/qml/MainView.qml src/plugins/qmlprofiler/qml/Overview.js src/plugins/qmlprofiler/qml/Overview.qml src/plugins/qmlprofiler/qml/TimeDisplay.qml src/plugins/qmlprofiler/qml/TimeMarks.qml src/plugins/qmlprofiler/qmlprofilertimelinemodelproxy.cpp src/plugins/qmlprofiler/sortedtimelinemodel.cpp src/plugins/qmlprofiler/sortedtimelinemodel.h src/plugins/qmlprofiler/timelinemodelaggregator.cpp src/plugins/qmlprofiler/timelinemodelaggregator.h src/plugins/qmlprofiler/timelinerenderer.cpp src/plugins/qmlprofiler/timelinerenderer.h src/plugins/qmlprojectmanager/QmlProjectManager.json.in src/plugins/texteditor/findinfiles.cpp src/plugins/vcsbase/vcsconfigurationpage.cpp src/shared/qbs src/shared/scriptwrapper/interface_wrap_helpers.h src/shared/scriptwrapper/wrap_helpers.h tests/auto/qmlprofiler/abstracttimelinemodel/tst_abstracttimelinemodel.cpp tests/system/suite_debugger/tst_debug_empty_main/test.py tests/system/suite_debugger/tst_qml_js_console/test.py tests/system/suite_debugger/tst_qml_locals/test.py Change-Id: I67540b648f8b162496f4aa606b04d50c7c9125c6
| * Update LicenseEike Ziller2015-01-161-6/+6
| | | | | | | | | | Change-Id: I711d5fb475ef814a1dc9d2822740e827f3f67125 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | Debugger: Pass RunControl in DebuggerStartParametershjk2015-02-101-1/+2
| | | | | | | | | | | | | | .. to simplify DebuggerRunControlFactory::doCreate() call. Change-Id: I4dd0c224968bb8a388ea7f095b940b66ee606ab1 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Valgrind: Suppress messagebox on SIGTRAP for vgdbOrgad Shaneh2015-02-101-0/+1
| | | | | | | | | | Change-Id: I5af3a3a67a9c9903337e0035733a74ad482ba197 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Valgrind: Support vgdbOrgad Shaneh2015-02-101-5/+78
| | | | | | | | | | Change-Id: Id9f653a81d329494017653b8fc7ec9960e20dbcd Reviewed-by: hjk <hjk@theqtcompany.com>
* | Valgrind: Port to Qt5-style connectOrgad Shaneh2015-02-091-2/+2
| | | | | | | | | | Change-Id: If5f36bb262b932b60133d4301ab614311ce1feee Reviewed-by: hjk <hjk@theqtcompany.com>
* | Misc: Remove unneeded qualificationsOrgad Shaneh2015-02-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mostly done using the following ruby script: Dir.glob('**/*.cpp').each { |file| next if file =~ %r{src/shared/qbs|/qmljs/} s = File.read(file) s.scan(/^using namespace (.*);$/) { ns = $1 t = s.gsub(/^(.*)\b#{ns}::((?!Const)[A-Z])/) { |m| before = $1 char = $2 if before =~ /"|\/\/|\\|using|SIGNAL|SLOT|Q_/ m else before + char end } if t != s puts file File.open(file, 'w').write(t) end } } Change-Id: I919da493d0629b719d328e5e71c96a29d230dfd1 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com> Reviewed-by: hjk <hjk@theqtcompany.com>
* | Port to new connect apiMontel Laurent2015-01-301-6/+6
|/ | | | | Change-Id: I873a36601d54065b61d0666558b93dc839ad0dfc Reviewed-by: hjk <hjk@theqtcompany.com>
* License updateEike Ziller2014-10-091-7/+8
| | | | | Change-Id: I3c22ef2685d7aa589f5d0ab74d693653a4c32082 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* Valgrind: Use generic timed future for uncertain progresshjk2014-05-131-9/+0
| | | | | Change-Id: Idb7f1b1e5a0fcd0351d6c30f5a6dadcbec191898 Reviewed-by: Eike Ziller <eike.ziller@digia.com>
* Valgrind: Remove unused function.Christian Kandeler2014-05-121-28/+0
| | | | | Change-Id: Ia14634a95e8227b65719052482dec9e9fd456877 Reviewed-by: hjk <hjk121@nokiamail.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>
* Remove most trailing newlines from translated messages.Friedemann Kleint2013-10-221-1/+1
| | | | | | | | | They are a hassle for translators and reviewers alike. Change-Id: I07c1b61e8b6719e54fdc1f69cf63f573119a6776 Reviewed-by: hjk <hjk121@nokiamail.com> Reviewed-by: Robert Loehning <robert.loehning@digia.com> Reviewed-by: Eike Ziller <eike.ziller@digia.com>
* Valgrind: Memcheck: Add gui options for --show-origins and --leak-checkNikolai Kosjar2013-09-101-0/+18
| | | | | Change-Id: If07c7a7a11a4c555e26b672bebb1e93eded38dc4 Reviewed-by: hjk <hjk121@nokiamail.com>
* TaskHub: Move code to be more conform with other singletonsTobias Hunger2013-08-191-5/+4
| | | | | | | | | | | Make methods static and add a instance() method for Signals/Slots. Remove ProjectExplorerPlugin::taskHub() method and use the new ones instead. Change-Id: Ifae24ff19579fc524cbd61bddc826095c443adfa Reviewed-by: Daniel Teske <daniel.teske@digia.com> Reviewed-by: hjk <hjk121@nokiamail.com>
* ProjectExplorer/Analyzer: Merge RunConfigurationAspectshjk2013-08-161-2/+0
| | | | | | | | The Analyzer implementation is now simple and still generic enough to serve as general base. Change-Id: I050a21919bf128929b77a64da1f46d157691d849 Reviewed-by: Daniel Teske <daniel.teske@digia.com>
* Analyzer: Cleanup settings handlinghjk2013-08-121-6/+5
| | | | | | | More flexible, less over-engineered this way. Change-Id: I3e224a6be85d3a187056d79fd506e8cf6a32c8a9 Reviewed-by: Daniel Teske <daniel.teske@digia.com>
* Analyzer: Merge IAnalyzerEngine and AnalyzerRunControlhjk2013-08-011-15/+25
| | | | | Change-Id: I74edaef59600a44924d2692c1ebc7f98d8581115 Reviewed-by: Christiaan Janssen <christiaan.janssen@digia.com>
* Analyzer: Simplify IAnalyzerEngine interfacehjk2013-08-011-2/+2
| | | | | | Change-Id: Id5eac0145e04b809bf535087d3bb9a04124c2e87 Reviewed-by: Aurindam Jana <aurindam.jana@digia.com> Reviewed-by: Christiaan Janssen <christiaan.janssen@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