diff options
author | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2022-04-16 14:13:59 +0200 |
---|---|---|
committer | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2022-04-17 23:21:19 +0200 |
commit | 8c55ce62b9e65533619870d1e5b72612a1a16406 (patch) | |
tree | a39beb02600048177920f3b888fed5b5f3fe66be /utils/rbutilqt/gui | |
parent | 1af92e5ff84bcb05d264d65d2904c4d9fe8f0d98 (diff) | |
download | rockbox-8c55ce62b9e65533619870d1e5b72612a1a16406.tar.gz rockbox-8c55ce62b9e65533619870d1e5b72612a1a16406.zip |
rbutil: Move bootloader file check to base class.
Avoid constructing the bootloader file path in the caller. Pass filename
list and mountpoint separately so it can directly fall back to the
mountpoint.
Change some functions to use references instead of creating temporary
objects.
Change-Id: I09c9d755553a32de3d02a42a8ce1fcb94f831b2a
Diffstat (limited to 'utils/rbutilqt/gui')
-rw-r--r-- | utils/rbutilqt/gui/selectiveinstallwidget.cpp | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/utils/rbutilqt/gui/selectiveinstallwidget.cpp b/utils/rbutilqt/gui/selectiveinstallwidget.cpp index 4417a52a29..7ee7498d64 100644 --- a/utils/rbutilqt/gui/selectiveinstallwidget.cpp +++ b/utils/rbutilqt/gui/selectiveinstallwidget.cpp | |||
@@ -308,17 +308,7 @@ void SelectiveInstallWidget::installBootloader(void) | |||
308 | // set bootloader filename. Do this now as installed() needs it. | 308 | // set bootloader filename. Do this now as installed() needs it. |
309 | QStringList blfile = PlayerBuildInfo::instance()->value( | 309 | QStringList blfile = PlayerBuildInfo::instance()->value( |
310 | PlayerBuildInfo::BootloaderFile).toStringList(); | 310 | PlayerBuildInfo::BootloaderFile).toStringList(); |
311 | QStringList blfilepath; | 311 | bl->setBlFile(RbSettings::value(RbSettings::Mountpoint).toString(), blfile); |
312 | for(int a = 0; a < blfile.size(); a++) { | ||
313 | blfilepath.append(RbSettings::value(RbSettings::Mountpoint).toString() | ||
314 | + blfile.at(a)); | ||
315 | } | ||
316 | // on devices without a bootloader file we use the mointpoint. The | ||
317 | // installer will use that to determine the correct device. | ||
318 | if(blfile.isEmpty()) { | ||
319 | blfilepath.append(RbSettings::value(RbSettings::Mountpoint).toString()); | ||
320 | } | ||
321 | bl->setBlFile(blfilepath); | ||
322 | QUrl url(PlayerBuildInfo::instance()->value(PlayerBuildInfo::BootloaderUrl).toString() | 312 | QUrl url(PlayerBuildInfo::instance()->value(PlayerBuildInfo::BootloaderUrl).toString() |
323 | + PlayerBuildInfo::instance()->value(PlayerBuildInfo::BootloaderName).toString()); | 313 | + PlayerBuildInfo::instance()->value(PlayerBuildInfo::BootloaderName).toString()); |
324 | bl->setBlUrl(url); | 314 | bl->setBlUrl(url); |