feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

Debugger: Add action do debugger tool bar

Browse Source 
The action for disabling or enabling all breakpoints
is quite hidden inside the context menu.
Make it more obvious by adding something similar to
the debugger tool bar.

Change-Id: Iedd1a03912eeca635ebc41b067ec785bb55d3474
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Christian Stenger
2024-07-30 14:14:41 +02:00
parent 04b9c08153
commit aed3eee06a
1 changed files with 23 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/plugins/debugger/debuggerengine.cpp
View File

@@ -512,6 +512,7 @@ public:
QAction m_abortAction{Tr::tr("Abort Debugging")}; QAction m_abortAction{Tr::tr("Abort Debugging")};
QAction m_stepIntoAction{Tr::tr("Step Into")}; QAction m_stepIntoAction{Tr::tr("Step Into")};
QAction m_stepOutAction{Tr::tr("Step Out")}; QAction m_stepOutAction{Tr::tr("Step Out")};
QAction m_toggleEnableBreakpointsAction{Tr::tr("Disable all Breakpoints")};
QAction m_runToLineAction{Tr::tr("Run to Line")}; // In the debug menu QAction m_runToLineAction{Tr::tr("Run to Line")}; // In the debug menu
QAction m_runToSelectedFunctionAction{Tr::tr("Run to Selected Function")}; QAction m_runToSelectedFunctionAction{Tr::tr("Run to Selected Function")};
QAction m_jumpToLineAction{Tr::tr("Jump to Line")}; QAction m_jumpToLineAction{Tr::tr("Jump to Line")};
@@ -794,6 +795,28 @@ void DebuggerEnginePrivate::setupViews()
connect(&m_watchAction, &QAction::triggered, connect(&m_watchAction, &QAction::triggered,
m_engine, &DebuggerEngine::handleAddToWatchWindow); m_engine, &DebuggerEngine::handleAddToWatchWindow);
m_toggleEnableBreakpointsAction.setIcon(Icons::BREAKPOINT_DISABLED.icon()); // FIXME better icon
m_toggleEnableBreakpointsAction.setCheckable(true);
m_perspective->addToolBarAction(&m_toggleEnableBreakpointsAction);
connect(&m_toggleEnableBreakpointsAction, &QAction::triggered,
m_engine, [this](bool checked) {
BreakHandler *handler = m_engine->breakHandler();
const auto bps = m_engine->breakHandler()->breakpoints();
for (const auto &bp : bps) {
if (auto gbp = bp->globalBreakpoint())
gbp->setEnabled(!checked, false);
handler->requestBreakpointEnabling(bp, !checked);
}
});
connect(m_engine->breakHandler(), &BreakHandler::dataChanged,
m_engine, [this] {
const auto bps = m_engine->breakHandler()->breakpoints();
const auto [enabled, disabled] = Utils::partition(bps, &BreakpointItem::isEnabled);
if (!enabled.isEmpty() && !disabled.isEmpty())
return;
m_toggleEnableBreakpointsAction.setChecked(!disabled.isEmpty());
});
m_perspective->addToolBarAction(&m_recordForReverseOperationAction); m_perspective->addToolBarAction(&m_recordForReverseOperationAction);
connect(&m_recordForReverseOperationAction, &QAction::triggered, connect(&m_recordForReverseOperationAction, &QAction::triggered,
m_engine, &DebuggerEngine::handleRecordReverse); m_engine, &DebuggerEngine::handleRecordReverse);