diff options
author | hjk <hjk@qt.io> | 2017-06-13 08:49:18 +0200 |
---|---|---|
committer | hjk <hjk@qt.io> | 2017-06-22 12:28:08 +0000 |
commit | ffc97df7a6c3add7820c08c1614d27d10a82a0d6 (patch) | |
tree | 12b25ddf66a640dd1c34ab2a87bbbe5d3724f5a9 /src/plugins/debugger/gdb/remotegdbserveradapter.cpp | |
parent | 107df0a7c1201bbb42d4f4d1e8b7bd705e7a310a (diff) |
Debugger: Remove remote setup sub-statemachinery
Not needed anymore in the world of RunWorkers.
Change-Id: Id7fb24fece6acb03de12f2677dd99a05c513e7a4
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io>
Diffstat (limited to 'src/plugins/debugger/gdb/remotegdbserveradapter.cpp')
-rw-r--r-- | src/plugins/debugger/gdb/remotegdbserveradapter.cpp | 43 |
1 files changed, 5 insertions, 38 deletions
diff --git a/src/plugins/debugger/gdb/remotegdbserveradapter.cpp b/src/plugins/debugger/gdb/remotegdbserveradapter.cpp index 4f50860134..a36aa11bef 100644 --- a/src/plugins/debugger/gdb/remotegdbserveradapter.cpp +++ b/src/plugins/debugger/gdb/remotegdbserveradapter.cpp @@ -28,7 +28,7 @@ #include <debugger/debuggeractions.h> #include <debugger/debuggercore.h> #include <debugger/debuggerprotocol.h> -#include <debugger/debuggerstartparameters.h> +#include <debugger/debuggerruncontrol.h> #include <debugger/procinterrupt.h> #include <coreplugin/messagebox.h> @@ -84,14 +84,10 @@ void GdbRemoteServerEngine::setupEngine() m_uploadProc.start(arglist); m_uploadProc.waitForStarted(); + m_uploadProc.waitForFinished(); } - if (runParameters().remoteSetupNeeded) { - notifyEngineRequestRemoteSetup(); - } else { - m_startAttempted = true; - startGdb(); - } + startGdb(); } void GdbRemoteServerEngine::uploadProcError(QProcess::ProcessError error) @@ -147,13 +143,9 @@ void GdbRemoteServerEngine::readUploadStandardError() void GdbRemoteServerEngine::uploadProcFinished() { if (m_uploadProc.exitStatus() == QProcess::NormalExit && m_uploadProc.exitCode() == 0) { - if (!m_startAttempted) - startGdb(); + // all good. } else { - RemoteSetupResult result; - result.success = false; - result.reason = m_uploadProc.errorString(); - notifyEngineRemoteSetupFinished(result); + runTool()->reportFailure(tr("Upload failed: %1").arg(m_uploadProc.errorString())); } } @@ -459,30 +451,5 @@ void GdbRemoteServerEngine::shutdownEngine() notifyAdapterShutdownOk(); } -void GdbRemoteServerEngine::notifyEngineRemoteServerRunning - (const QString &serverChannel, int inferiorPid) -{ - // Currently only used by Android support. - runParameters().attachPID = Utils::ProcessHandle(inferiorPid); - runParameters().remoteChannel = serverChannel; - runParameters().useExtendedRemote = true; - showMessage("NOTE: REMOTE SERVER RUNNING IN MULTIMODE"); - m_startAttempted = true; - startGdb(); -} - -void GdbRemoteServerEngine::notifyEngineRemoteSetupFinished(const RemoteSetupResult &result) -{ - QTC_ASSERT(state() == EngineSetupRequested, qDebug() << state()); - GdbEngine::notifyEngineRemoteSetupFinished(result); - - if (result.success) { - if (!m_startAttempted) - startGdb(); - } else { - handleAdapterStartFailed(result.reason); - } -} - } // namespace Internal } // namespace Debugger |