diff options
| author | Tom Tanner <thosrtanner2@users.sourceforge.net> | 2015-08-13 22:37:33 +0100 |
|---|---|---|
| committer | Tom Tanner <thosrtanner2@users.sourceforge.net> | 2015-08-13 22:37:33 +0100 |
| commit | 821f95b8ab259f89f379c10e533c42476c56d361 (patch) | |
| tree | cbe6388e85de04924a27762e87705248fe361f0d /src/mainwindow.cpp | |
| parent | 716dbcc97d306115df4bcd32cef717e616190ce4 (diff) | |
Clean up of fixes for custom executable window
Diffstat (limited to 'src/mainwindow.cpp')
| -rw-r--r-- | src/mainwindow.cpp | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp index 28596127..a0a56406 100644 --- a/src/mainwindow.cpp +++ b/src/mainwindow.cpp @@ -1755,8 +1755,7 @@ void MainWindow::on_executablesListBox_currentIndexChanged(int index) if (executablesList->isEnabled()) {
//I think the 2nd test is impossible
- if (index == 0 ||
- index > static_cast<int>(m_OrganizerCore.executablesList()->size())) {
+ if ((index == 0) || (index > static_cast<int>(m_OrganizerCore.executablesList()->size()))) {
if (modifyExecutablesDialog()) {
setExecutableIndex(previousIndex);
}
@@ -3563,13 +3562,13 @@ void MainWindow::addAsExecutable() targetInfo.baseName());
if (!name.isEmpty()) {
//Note: If this already exists, you'll lose custom settings
- m_OrganizerCore.executablesList()->configureExecutable(name,
- binaryInfo.absoluteFilePath(),
- arguments,
- targetInfo.absolutePath(),
- ExecutableInfo::CloseMOStyle::DEFAULT_STAY,
- QString(),
- Executable::CustomExecutable);
+ m_OrganizerCore.executablesList()->addExecutable(name,
+ binaryInfo.absoluteFilePath(),
+ arguments,
+ targetInfo.absolutePath(),
+ ExecutableInfo::CloseMOStyle::DEFAULT_STAY,
+ QString(),
+ Executable::CustomExecutable);
refreshExecutablesList();
}
} break;
|