summaryrefslogtreecommitdiff
path: root/src/plugins/valgrind/callgrindtool.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Debugger: Dissolve AnalyzerUtils::logToIssuesPane()hjk2016-05-311-2/+4
| | | | | | | | | If the Error + popup combo is generic it should live in TaskHub. If it is not, end user code should decide when to popup. Not an intermediate level. Change-Id: I195ba8c17c5cc192ff9762ed6f45629143bcbb35 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Icons: Move "next/prev", "[un]locked" and "zoom" from Core to UtilsAlessandro Portale2016-05-311-2/+3
| | | | | | | | | | They are needed in the timeline view. The change in qtcreator-project.qdocconf makes sure that qdoc still finds the referenced icons. Change-Id: I812fba8e57d06d93efc00c295467c83c08cd1784 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* Debugger: Move remaining Analyzer overlay icon to ProjectExplorerhjk2016-05-301-2/+1
| | | | | | | | It's the only remaining item and loosens the dependencies of ex-AnalyzerBase based plugins on the the Debugger. Change-Id: I943ac44401c440dd6d3d5c1f54f8f996accd2b4a Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* Debugger: Let perspectives specify their central widgethjk2016-05-131-2/+2
| | | | | | | | | | | The default 0 value will be interpreted as 'use the editor stack'. Also, drop the idea of value semantics for Perspective objects to get a simpler approach to the destruction of owned widgets (tools docks + central widget) Change-Id: Ic6470411ee5d387c43447f95b5a12c81c6658ff8 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Valgrind: Remove factories from object pool when destroying themUlf Hermann2016-05-111-1/+1
| | | | | | | Otherwise we get crashes on exit. Change-Id: I806f374ba8f835c9e4361599aa901cfc4b2f0720 Reviewed-by: hjk <hjk@theqtcompany.com>
* Valgrind: Split MemCheck and CallGrind factorieshjk2016-05-101-10/+58
| | | | | | | | 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>
* Theme: Separate run/stop/interrupt icons in menusAlessandro Portale2016-04-211-3/+3
| | | | | | | And for that, we also need separate theme color roles. Change-Id: I27bf4499acb516f7f824f5e57632797653b150ec Reviewed-by: hjk <hjk@theqtcompany.com>
* Valgrind: Avoid soft assert on missing line numbersChristian Stenger2016-04-151-0/+3
| | | | | | | | | If callgrind does not find a valid line number information trying to paint the respective text mark runs into a soft assert. Avoid this by not creating a text mark for such results. Change-Id: Ib1c49db82fd4d46bf84ecdc398b88fb0ca09707d Reviewed-by: hjk <hjk@theqtcompany.com>
* Icons: Move Run/Stop/Interrupt from projectexplorer to coreAlessandro Portale2016-04-101-2/+1
| | | | | Change-Id: Iba65c2ede538049c9b203fd4c2f99ef552019862 Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* Valgrind: Fix UI textLeena Miettinen2016-04-051-4/+4
| | | | | | | | | | - Fix typo in product name - Fix grammar issue - Use standard phrasing - Fix capitalization Change-Id: I0456c60943b51b19d5f03fb2908a7cc5af1f3595 Reviewed-by: hjk <hjk@theqtcompany.com>
* Valgrind: Fix callgrind's "Cost format" buttonAlessandro Portale2016-03-311-6/+1
| | | | | | | | | Broke after 0a89b89065b633cfb4f5dc8c9fd38240b38e53a8 Correct fix seems to me to avoid the whole menu. Change-Id: I59f1eb859c9045fc5ae9ac1b377b7e69b29af7ae Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Valgrind: Fix memory leaksOrgad Shaneh2016-03-261-0/+1
| | | | | | | Detected by memcheck. Change-Id: I7eaf4cf7ee2b4c03b03bfc1bbc2d49c68612b6ad Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Fix object leakage on shutdown and heap-use-after-free.hjk2016-03-241-1/+0
| | | | | | Task-number: QTCREATORBUG-15938 Change-Id: I437756705c33730398a129651fabe34c92334656 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Valgrind: Disable start actions during buildinghjk2016-03-101-10/+5
| | | | | | | This prevents multiple starts of the same tool. Change-Id: I2df89fa4336ff649a56c4dfb1f721ac31497d396 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Merge debug mode and analyze modehjk2016-03-071-264/+265
| | | | | | | | | | | | | | | | | | | | | | | | 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>
* Valgrind: Shorten and correct start action namehjk2016-03-071-1/+1
| | | | | | | | | The applications to attach to do not have to be remote at all. Also, the name should be as short as possible as space in the status bar is not unlimited. Change-Id: I1df8e025114cd8ad94daa5219c26e61ce70e5e5d Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>
* Debugger: Make Perspective status label globalhjk2016-03-021-3/+3
| | | | | | | | After some discussion we agreed that the contents is ephemeral and does not need to survive perspective switching. Change-Id: I41de6a8f9478e4bd229c8b204ef7a3fa0a344b75 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Valgrind: Clean up tools interface a bithjk2016-03-021-272/+281
| | | | | | | | Mainly move tool specific code from plugin.cpp to *tool.cpp. Clean up includes etc. Change-Id: Ic968ead9d93099c59abe9f99a9db529305160f95 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Remove return value from AnalyzerManager::createDockWidget()hjk2016-03-021-15/+15
| | | | | Change-Id: If6cd2a68f2f99a1977ded0f05a12790194143274 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Valgrind: Avoid soft assert when no project is openedChristian Stenger2016-02-291-2/+1
| | | | | Change-Id: I3ff8a3dbc3b6115824477e307c03d9a2c0d19285 Reviewed-by: hjk <hjk@theqtcompany.com>
* Analyzer: Rework the dock widget layout generationhjk2016-02-291-21/+18
| | | | | | | | | | Decouple layout generation from widget generation and separate analyzer action description from menu action creation. Tool specific layouts are named "Perspective" now. Change-Id: I774efe77a07640c4cc26e4e566662c8a673c8831 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Move analyzerbase to debuggerhjk2016-02-251-3/+3
| | | | | | | | | | | | This is the first mechanical step to execute on the 'shared pool of debugger/analyzer views' idea. Future steps would be providing infrastructure for the view pool, making all analyzer/debugger views use the pool and then re-extract a sensible base for a 'analyzer-and/or-debugger' tool plugin interface. Change-Id: I1bb392e6dd3084fc56937956bee1d6fd9530335d Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Valgrind: Random cleanupshjk2016-01-211-2/+2
| | | | | | | Remove unneeded function arguments, Qt 5 connects etc. Change-Id: I95faf80e7b5ccc574e2457b841f7913bc2aa05d3 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Remove AnalyzerStartParameters from AnalyzerRunControl ctorhjk2016-01-201-8/+5
| | | | | Change-Id: I0f7e982a9144469869e0c42daedfb411789511ca 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>
* Core: Unify the close iconsAlessandro Portale2015-12-041-1/+1
| | | | | | | Use the same image file and recolor it as needed, Change-Id: I914819a53ca05d83ca5b88adf44706b460b48ac0 Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* ProjectExplorer: Themed pause buttonAlessandro Portale2015-12-031-1/+2
| | | | | Change-Id: I61a678ec3696a5d0a0555e342fb34604fc1a5cdd Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* Themed Icons: Introduce Utils::IconAlessandro Portale2015-11-251-9/+8
| | | | | | | | | | | | | | Instead of describing icons via file name or in the themed icons case via a string that is a list of mask/color pairs, we have now a class for it. Icons are now listed in per-plugin *icons.h headers. RunControl::m_icon was The only place left where an icon property was in fact a string. This patch changes that member to be a Utils::Icon. Change-Id: Ibcfa8bb25e6d2e330c567ee7ccc0b97ead603177 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Icon refresh: First step towards the new Qt Creator themesAlessandro Portale2015-10-231-3/+4
| | | | | | | | | | http://blog.qt.io/blog/author/didesous/ announced new designs/themes for Qt Creator. This patch replaces many of the existing toolbar icons with recolorizable masks for better theming support. Change-Id: I557aa485205fe2624f33724226f698c303342b40 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com> Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* Valgrind: Show status messages permanently.Christian Kandeler2015-07-071-3/+3
| | | | | | | There is no reason for these messages to fade away after a while. Change-Id: I6dcf199af87df77b6d65e267a45f5553c85bc952 Reviewed-by: hjk <hjk@theqtcompany.com>
* Analyzer: Have one status label per tool.Christian Kandeler2015-07-011-3/+3
| | | | | | | | It does not make sense for all the analyzers to share a single status label, as they are unrelated to each other. Change-Id: I9cf885263853251f841ef96836860455905677ac Reviewed-by: hjk <hjk@theqtcompany.com>
* Analyzer: Move towards less global statehjk2015-06-231-2/+1
| | | | | | | Make startTool() explicit as selectAction(Id) + startAction(Id). Change-Id: I695112bd8419021a781234121a55440b5dc381be Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com>
* Valgrind: Start custom actions more directlyhjk2015-06-191-3/+3
| | | | | | | This does not need to go through the PE machinery. Change-Id: If0b2fde5309d7e119e8ac27245a22939b68d847d Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Move tool startup code closer to the action setuphjk2015-06-181-18/+0
| | | | | | | | | The code sharing in the base was along a somewhat unfortunate boundary. Moving it into the only user also removes some of the wrong local/remote separation. Change-Id: I0be9636ea448d123f9f5105a52d56f47ff7871c3 Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com>
* Valgrind: Log errors to issues pane instead of popping up a QMessageBox.Christian Kandeler2015-03-031-3/+2
| | | | | | | | Just as informative, but less intrusive and can't introduce event loop problems. Change-Id: I7dec17b1e82ba23340376075125f965017047959 Reviewed-by: hjk <hjk@theqtcompany.com>
* Clean exported headers of the TextEditor plugin.Friedemann Kleint2015-02-261-0/+2
| | | | | Change-Id: I1e7dd34ba5a51fb0b34d137dc03add4457b32ed1 Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* ProjectExplorer: Add runStartupProject() convenience functionhjk2015-02-231-2/+1
| | | | | | | | The usage pattern seems to be used often enough to justify a function. Change-Id: Iecdeb2b42879f419154902e685d0d13e8cae8b33 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com> Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
* Analyzer: Reduce explicit use of StartMode enumhjk2015-02-201-4/+7
| | | | | Change-Id: I27b1d06395dea940c8dd39cd2bd41fc09cee3a9c Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzers: Various cleanupshjk2015-02-191-1/+1
| | | | | | | Fallout after the demise of IAnalizerTool. Change-Id: I68f936ed5f2929dc4f09a408bdf6b0ea38e568ef Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Valgrind: Simplify callgrind context menu creationhjk2015-02-191-23/+4
| | | | | Change-Id: I5ea4bdf2dc6f4d3867e6abbc8d4cd6a1149a17b2 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Dissolve IAnalyzerToolhjk2015-02-181-6/+6
| | | | | Change-Id: I1fd90cf00d6b9dff54b4555a14747282830a4b93 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Move non-static parts of IAnalyzerTool::start*Toolhjk2015-02-181-8/+17
| | | | | | | ... to user code. Change-Id: Ibe278134e39cfb3f90acedc976fab5e9dc019046 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Remove IAnalyzerTool::m_toolMode memberhjk2015-02-181-2/+1
| | | | | | | It's always known from the context when needed. Change-Id: Ibf314dfb779190e89b775acab784ed71acff2c83 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Analyzer: Move runMode member from IAnalyzerTool to AnalyzerActionhjk2015-02-181-3/+2
| | | | | | | Slim down tool before dissolving. Change-Id: I502aaa8860293acf8bfd93f541b751d5b96a9563 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Valgrind: Dissolve ValgrindTool hierarchy levelhjk2015-02-181-2/+5
| | | | | | | | Medium term all *Tool should be replaced by RunControl(Factory) to make the analyzer architecture more similar to Qt Creator core Change-Id: I892cfc7fe45c73cc6ac4442a288810b83cb24c17 Reviewed-by: Christian Stenger <christian.stenger@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>
* | Misc: Remove unneeded qualificationsOrgad Shaneh2015-02-061-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | Callgrind: Convert to Qt5-style connectOrgad Shaneh2015-02-041-41/+44
| | | | | | | | | | Change-Id: Id8851f31cf3a4005d7d4573bfb3dc9093932a4c2 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Core: Use FileName for file path in IDocumentOrgad Shaneh2015-01-121-1/+1
| | | | | | | | | | Change-Id: I85f7398aee59d0d36f0e5c3bf88ff3c96002e394 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
View Lorry Controller Status