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

Debugger: Transform DebuggerRunParameters::expectedSignals

Browse Source 
Task-number: QTCREATORBUG-29168
Change-Id: I5f123581a70979ea35e5aabc76fb10af4813c468
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Jarek Kobus
2025-01-10 13:05:53 +01:00
parent bdc1b186c1
commit 519446e05b
5 changed files with 6 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/plugins/debugger/debuggerengine.h
View File

@@ -138,8 +138,8 @@ public:
void setCommandsAfterConnect(const QString &commands) { m_commandsAfterConnect = commands; }
QString commandsAfterConnect() const { return m_commandsAfterConnect; }
// Used by Valgrind
QStringList expectedSignals;
void addExpectedSignal(const QString &signal) { m_expectedSignals.append(signal); }
QStringList expectedSignals() const { return m_expectedSignals; }
// For QNX debugging
bool useCtrlCStub = false;
@@ -244,6 +244,8 @@ private:
QString m_commandsForReset; // Used by baremetal plugin. Commands used for resetting the inferior
bool m_useContinueInsteadOfRun = false; // If connected to a hw debugger run is not possible but continue is used
QString m_commandsAfterConnect; // Additional commands to post after connection to debug target
QStringList m_expectedSignals; // Used by Valgrind
};
namespace Internal {

5
src/plugins/debugger/debuggerruncontrol.cpp
View File

@@ -204,11 +204,6 @@ void DebuggerRunTool::setCrashParameter(const QString &event)
m_runParameters.crashParameter = event;
}
void DebuggerRunTool::addExpectedSignal(const QString &signal)
{
m_runParameters.expectedSignals.append(signal);
}
void DebuggerRunTool::addSearchDirectory(const Utils::FilePath &dir)
{
m_runParameters.additionalSearchDirectories.append(dir);

2
src/plugins/debugger/debuggerruncontrol.h
View File

@@ -33,8 +33,6 @@ public:
void setStartMessage(const QString &msg);
void setCrashParameter(const QString &event);
void addExpectedSignal(const QString &signal);
void setSysRoot(const Utils::FilePath &sysRoot);
void setContinueAfterAttach(bool on);

2
src/plugins/debugger/gdb/gdbengine.cpp
View File

@@ -1421,7 +1421,7 @@ void GdbEngine::handleStop2(const GdbMi &data)
QString meaning = data["signal-meaning"].data();
// Ignore these as they are showing up regularly when
// stopping debugging.
if (name == stopSignal(rp.toolChainAbi) || rp.expectedSignals.contains(name)) {
if (name == stopSignal(rp.toolChainAbi) || rp.expectedSignals().contains(name)) {
showMessage(name + " CONSIDERED HARMLESS. CONTINUING.");
} else if (m_isQnxGdb && name == "0" && meaning == "Signal 0") {
showMessage("SIGNAL 0 CONSIDERED BOGUS.");

2
src/plugins/valgrind/memchecktool.cpp
View File

@@ -193,7 +193,7 @@ void MemcheckToolRunner::startDebugger(qint64 valgrindPid)
rp.setDisplayName(QString("VGdb %1").arg(valgrindPid));
rp.setRemoteChannel(QString("| vgdb --pid=%1").arg(valgrindPid));
rp.setUseContinueInsteadOfRun(true);
debugger->addExpectedSignal("SIGTRAP");
rp.addExpectedSignal("SIGTRAP");
connect(runControl(), &RunControl::stopped, debugger, &RunControl::deleteLater);