debugger: simplify the handling of the start script option

Script will be used if and only if there is one given. No extra bool.

Change-Id: I095e01b2243f0fa1f7ba2ce9591e3d67b6de5241
Reviewed-by: hjk <qthjk@ovi.com>

src/plugins/android/androiddebugsupport.cpp

@@ -81,8 +81,6 @@ RunControl *AndroidDebugSupport::createDebugRunControl(AndroidRunConfiguration *
             params.toolChainAbi = tc->targetAbi();
         params.executable = project(runConfig)->rootQt4ProjectNode()->buildDir() + QLatin1String("/app_process");
         params.remoteChannel = runConfig->remoteChannel();
-        params.useServerStartScript = true;
-
         params.solibSearchPath.clear();
         QList<Qt4ProFileNode *> nodes = project(runConfig)->allProFiles();
         foreach (Qt4ProFileNode *node, nodes)

src/plugins/debugger/debuggerdialogs.cpp

@@ -95,7 +95,6 @@ public:
     QCheckBox *breakAtMainCheckBox;
     QCheckBox *runInTerminalCheckBox;
     PathChooser *debuginfoPathChooser;
-    QCheckBox *useServerStartScriptCheckBox;
     QLabel *serverStartScriptLabel;
     PathChooser *serverStartScriptPathChooser;
     QComboBox *historyComboBox;
@@ -134,13 +133,12 @@ public:
     QString workingDirectory;
     bool breakAtMain;
     bool runInTerminal;
-    bool useServerStartScript;
     QString serverStartScript;
     QString debugInfoLocation;
 };
 
 StartApplicationParameters::StartApplicationParameters() :
-    breakAtMain(false), runInTerminal(false), useServerStartScript(false)
+    breakAtMain(false), runInTerminal(false)
 {
 }
 
@@ -151,7 +149,6 @@ bool StartApplicationParameters::equals(const StartApplicationParameters &rhs) c
         && workingDirectory == rhs.workingDirectory
         && breakAtMain == rhs.breakAtMain
         && runInTerminal == rhs.runInTerminal
-        && useServerStartScript == rhs.useServerStartScript
         && serverStartScript == rhs.serverStartScript
         && profileId == rhs.profileId
         && debugInfoLocation == rhs.debugInfoLocation;
@@ -184,7 +181,6 @@ void StartApplicationParameters::toSettings(QSettings *settings) const
     settings->setValue(_("LastExternalWorkingDirectory"), workingDirectory);
     settings->setValue(_("LastExternalBreakAtMain"), breakAtMain);
     settings->setValue(_("LastExternalRunInTerminal"), runInTerminal);
-    settings->setValue(_("LastUseServerStartScript"), useServerStartScript);
     settings->setValue(_("LastServerStartScript"), serverStartScript);
     settings->setValue(_("LastDebugInfoLocation"), debugInfoLocation);
 }
@@ -199,7 +195,6 @@ void StartApplicationParameters::fromSettings(const QSettings *settings)
     breakAtMain = settings->value(_("LastExternalBreakAtMain")).toBool();
     runInTerminal = settings->value(_("LastExternalRunInTerminal")).toBool();
     serverStartScript = settings->value(_("LastServerStartScript")).toString();
-    useServerStartScript = settings->value(_("LastUseServerStartScript")).toBool();
     debugInfoLocation = settings->value(_("LastDebugInfoLocation")).toString();
 }
 
@@ -239,15 +234,22 @@ StartApplicationDialog::StartApplicationDialog(QWidget *parent)
     d->breakAtMainCheckBox = new QCheckBox(this);
     d->breakAtMainCheckBox->setText(QString());
 
-    d->debuginfoPathChooser = new PathChooser(this);
-    d->debuginfoPathChooser->setPromptDialogTitle(tr("Select Location of Debugging Information"));
-
-    d->useServerStartScriptCheckBox = new QCheckBox(this);
     d->serverStartScriptPathChooser = new PathChooser(this);
     d->serverStartScriptPathChooser->setExpectedKind(PathChooser::File);
     d->serverStartScriptPathChooser->setPromptDialogTitle(tr("Select Server Start Script"));
+    d->serverStartScriptPathChooser->setToolTip(tr(
+        "This option can be used to point to a script that will be used "
+        "to start a debug server. If the field is empty, Qt Creator's "
+        "default methods to set up debug servers will be used."));
     d->serverStartScriptLabel = new QLabel(tr("&Server start script:"), this);
     d->serverStartScriptLabel->setBuddy(d->serverStartScriptPathChooser);
+    d->serverStartScriptLabel->setToolTip(d->serverStartScriptPathChooser->toolTip());
+
+    d->debuginfoPathChooser = new PathChooser(this);
+    d->debuginfoPathChooser->setPromptDialogTitle(tr("Select Location of Debugging Information"));
+    d->debuginfoPathChooser->setToolTip(tr(
+        "Base path for external debug information and debug sources."
+        "If empty, $SYSROOT/usr/lib/debug will be chosen."));
 
     QFrame *line = new QFrame(this);
     line->setFrameShape(QFrame::HLine);
@@ -271,21 +273,18 @@ StartApplicationDialog::StartApplicationDialog(QWidget *parent)
     formLayout->addRow(tr("&Working directory:"), d->workingDirectory);
     formLayout->addRow(tr("Run in &terminal:"), d->runInTerminalCheckBox);
     formLayout->addRow(tr("Break at \"&main\":"), d->breakAtMainCheckBox);
-    formLayout->addRow(tr("Use external script to start debug server"), d->useServerStartScriptCheckBox);
     formLayout->addRow(d->serverStartScriptLabel, d->serverStartScriptPathChooser);
-    formLayout->addRow(tr("Location of debugging &information:"), d->debuginfoPathChooser);
-    formLayout->addRow(line);
-    formLayout->addRow(tr("&Recent:"), d->historyComboBox);
+    formLayout->addRow(tr("Debug &information:"), d->debuginfoPathChooser);
     formLayout->addRow(line2);
+    formLayout->addRow(tr("&Recent:"), d->historyComboBox);
 
     QVBoxLayout *verticalLayout = new QVBoxLayout(this);
     verticalLayout->addLayout(formLayout);
-    verticalLayout->addItem(new QSpacerItem(0, 0, QSizePolicy::Minimum, QSizePolicy::Expanding));
-    verticalLayout->addWidget(line);
+    verticalLayout->addStretch();
+    verticalLayout->addRow(line);
     verticalLayout->addWidget(d->buttonBox);
 
     connect(d->localExecutablePathChooser, SIGNAL(changed(QString)), SLOT(updateState()));
-    connect(d->useServerStartScriptCheckBox, SIGNAL(toggled(bool)), SLOT(updateState()));
     connect(d->buttonBox, SIGNAL(accepted()), SLOT(accept()));
     connect(d->buttonBox, SIGNAL(rejected()), SLOT(reject()));
     connect(d->historyComboBox, SIGNAL(currentIndexChanged(int)),
@@ -309,11 +308,10 @@ void StartApplicationDialog::setHistory(const QList<StartApplicationParameters>
     }
 }
 
-void StartApplicationDialog::setScriptVisible(bool on)
+void StartApplicationDialog::hideStartScript()
 {
-    d->useServerStartScriptCheckBox->setVisible(on);
-    d->serverStartScriptPathChooser->setVisible(on);
-    d->serverStartScriptLabel->setVisible(on);
+    d->serverStartScriptPathChooser->setVisible(false);
+    d->serverStartScriptLabel->setVisible(false);
 }
 
 void StartApplicationDialog::historyIndexChanged(int index)
@@ -334,9 +332,6 @@ void StartApplicationDialog::updateState()
 {
     bool okEnabled = d->localExecutablePathChooser->isValid();
     d->buttonBox->button(QDialogButtonBox::Ok)->setEnabled(okEnabled);
-    bool startScriptEnabled = d->useServerStartScriptCheckBox->isChecked();
-    d->serverStartScriptLabel->setEnabled(startScriptEnabled);
-    d->serverStartScriptPathChooser->setEnabled(startScriptEnabled);
 }
 
 bool StartApplicationDialog::run(QWidget *parent, QSettings *settings, DebuggerStartParameters *sp)
@@ -362,7 +357,8 @@ bool StartApplicationDialog::run(QWidget *parent, QSettings *settings, DebuggerS
     StartApplicationDialog dialog(parent);
     dialog.setHistory(history);
     dialog.setParameters(history.back());
-    dialog.setScriptVisible(sp->startMode != AttachToRemoteServer);
+    if (sp->startMode == AttachToRemoteServer)
+        dialog.hideStartScript();
     if (dialog.exec() != QDialog::Accepted)
         return false;
 
@@ -391,7 +387,6 @@ bool StartApplicationDialog::run(QWidget *parent, QSettings *settings, DebuggerS
     if (!newParameters.processArgs.isEmpty())
         sp->processArgs = newParameters.processArgs;
     sp->breakOnMain = newParameters.breakAtMain;
-    sp->useServerStartScript = newParameters.useServerStartScript;
     sp->serverStartScript = newParameters.serverStartScript;
     sp->debugInfoLocation = newParameters.debugInfoLocation;
 
@@ -405,7 +400,6 @@ StartApplicationParameters StartApplicationDialog::parameters() const
 {
     StartApplicationParameters result;
     result.localExecutable = d->localExecutablePathChooser->path();
-    result.useServerStartScript = d->useServerStartScriptCheckBox->isChecked();
     result.serverStartScript = d->serverStartScriptPathChooser->path();
     result.profileId = d->profileChooser->currentProfileId();
     result.debugInfoLocation = d->debuginfoPathChooser->path();
@@ -420,7 +414,6 @@ void StartApplicationDialog::setParameters(const StartApplicationParameters &p)
 {
     d->profileChooser->setCurrentProfileId(p.profileId);
     d->localExecutablePathChooser->setPath(p.localExecutable);
-    d->useServerStartScriptCheckBox->setChecked(p.useServerStartScript);
     d->serverStartScriptPathChooser->setPath(p.serverStartScript);
     d->debuginfoPathChooser->setPath(p.debugInfoLocation);
     d->arguments->setText(p.processArgs);

src/plugins/debugger/debuggerdialogs.h

@@ -76,7 +76,7 @@ private:
     StartApplicationParameters parameters() const;
     void setParameters(const StartApplicationParameters &p);
     void setHistory(const QList<StartApplicationParameters> &l);
-    void setScriptVisible(bool on);
+    void hideStartScript();
     Core::Id profileId() const;
 
     StartApplicationDialogPrivate *d;

src/plugins/debugger/debuggerengine.cpp

@@ -127,7 +127,6 @@ QDebug operator<<(QDebug str, const DebuggerStartParameters &sp)
             << " useTerminal=" << sp.useTerminal
             << " remoteChannel=" << sp.remoteChannel
             << " symbolFileName=" << sp.symbolFileName
-            << " useServerStartScript=" << sp.useServerStartScript
             << " serverStartScript=" << sp.serverStartScript
             << " abi=" << sp.toolChainAbi.toString() << '\n';
     return str;

src/plugins/debugger/debuggerplugin.cpp

@@ -1567,7 +1567,6 @@ void DebuggerPluginPrivate::attachToRemoteServer()
     sp.startMode = AttachToRemoteServer;
     if (StartApplicationDialog::run(mainWindow(), m_coreSettings, &sp)) {
         sp.closeMode = KillAtClose;
-        sp.useServerStartScript = false;
         sp.serverStartScript.clear();
         if (RunControl *rc = createDebugger(sp))
             startDebugger(rc);
@@ -2576,8 +2575,6 @@ static QString formatStartParameters(DebuggerStartParameters &sp)
     str << "Sysroot: " << sp.sysRoot << '\n';
     str << "Debug Source Location: " << sp.debugSourceLocation.join(QLatin1String(":")) << '\n';
     str << "Symbol file: " << sp.symbolFileName << '\n';
-    if (sp.useServerStartScript)
-        str << "Using server start script: " << sp.serverStartScript;
     str << "Dumper libraries: " << QDir::toNativeSeparators(sp.dumperLibrary);
     foreach (const QString &dl, sp.dumperLibraryLocations)
         str << ' ' << QDir::toNativeSeparators(dl);

src/plugins/debugger/debuggerstartparameters.h

@@ -65,7 +65,6 @@ public:
         languages(AnyLanguage),
         qmlServerAddress(QLatin1String("127.0.0.1")),
         qmlServerPort(ProjectExplorer::Constants::QML_DEFAULT_DEBUG_SERVER_PORT),
-        useServerStartScript(false),
         remoteSetupNeeded(false),
         startMode(NoStartMode),
         closeMode(KillAtClose),
@@ -112,7 +111,6 @@ public:
     // Used by remote debugging.
     QString remoteChannel;
     QString symbolFileName;
-    bool useServerStartScript;
     QString serverStartScript;
     QString searchPath; // Gdb "set solib-search-path"
     QString debugInfoLocation; // Gdb "set-debug-file-directory".

src/plugins/debugger/gdb/remotegdbserveradapter.cpp

@@ -86,13 +86,9 @@ void GdbRemoteServerEngine::setupEngine()
 {
     QTC_ASSERT(state() == EngineSetupRequested, qDebug() << state());
     showMessage(_("TRYING TO START ADAPTER"));
-    if (startParameters().useServerStartScript) {
-        if (startParameters().serverStartScript.isEmpty()) {
-            showMessage(_("No server start script given. "), StatusBar);
-        } else {
-            m_uploadProc.start(_("/bin/sh ") + startParameters().serverStartScript);
-            m_uploadProc.waitForStarted();
-        }
+    if (!startParameters().serverStartScript.isEmpty()) {
+        m_uploadProc.start(_("/bin/sh ") + startParameters().serverStartScript);
+        m_uploadProc.waitForStarted();
     }
     if (startParameters().remoteSetupNeeded)
         notifyEngineRequestRemoteSetup();

src/plugins/debugger/gdb/startgdbserverdialog.cpp

@@ -213,9 +213,6 @@ void GdbServerStarter::attach(int port)
     sp.executable = localExecutable;
     sp.startMode = AttachToRemoteServer;
     sp.closeMode = KillAtClose;
-    sp.overrideStartScript.clear();
-    sp.useServerStartScript = false;
-    sp.serverStartScript.clear();
     sp.sysRoot = SysRootProfileInformation::sysRoot(d->profile).toString();
     sp.debuggerCommand = DebuggerProfileInformation::debuggerCommand(d->profile).toString();
     sp.connParams = d->device->sshParameters();

src/plugins/debugger/loadcoredialog.cpp

@@ -290,7 +290,7 @@ AttachCoreDialog::AttachCoreDialog(QWidget *parent)
 
     QVBoxLayout *vboxLayout = new QVBoxLayout(this);
     vboxLayout->addLayout(formLayout);
-    vboxLayout->addSpacerItem(new QSpacerItem(0, 0, QSizePolicy::Minimum, QSizePolicy::Expanding));
+    vboxLayout->addStretch();
     vboxLayout->addWidget(line);
     vboxLayout->addWidget(d->buttonBox);