mirror of
https://github.com/Rockbox/rockbox.git
synced 2025-10-13 18:17:39 -04:00
plugin load / tagtree retrieval: eliminate storage_disk_is_active check
Since commit e0df995
storage_disk_is_active may be 0
even if there is no disk that needs to be spun up or
device to power on, so it doesn't really make sense
to check for it, anymore, in order to decide whether
to display the loading indicator.
For remaining devices with spinning disks, the noise
or vibration of the disk spinning up may serve as a
sufficient signal when loading a plugin, that the
device isn't frozen. Plus, for the tagtree, search
progress is displayed after 0.5s anyway.
Change-Id: I5791725e2388d7113818a358204968d8a4e6f843
This commit is contained in:
parent
b8238f7b20
commit
ef19fa4408
2 changed files with 2 additions and 16 deletions
|
@ -882,10 +882,6 @@ int plugin_load(const char* plugin, const void* parameter)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef HAVE_DISK_STORAGE
|
|
||||||
if (!storage_disk_is_active())
|
|
||||||
splash(0, ID2P(LANG_WAIT));
|
|
||||||
#endif
|
|
||||||
strcpy(current_plugin, plugin);
|
strcpy(current_plugin, plugin);
|
||||||
current_plugin_handle = lc_open(plugin, pluginbuf, PLUGIN_BUFFER_SIZE);
|
current_plugin_handle = lc_open(plugin, pluginbuf, PLUGIN_BUFFER_SIZE);
|
||||||
if (current_plugin_handle == NULL) {
|
if (current_plugin_handle == NULL) {
|
||||||
|
|
|
@ -1465,18 +1465,8 @@ static int retrieve_entries(struct tree_context *c, int offset, bool init)
|
||||||
int sort_limit;
|
int sort_limit;
|
||||||
int strip;
|
int strip;
|
||||||
|
|
||||||
/* Show search progress straight away if the disk needs to spin up,
|
/* Show search progress after 0.5s delay */
|
||||||
otherwise show it after the normal 1/2 second delay */
|
show_search_progress(true, 0, 0, 0);
|
||||||
show_search_progress(
|
|
||||||
#ifdef HAVE_DISK_STORAGE
|
|
||||||
#ifdef HAVE_TC_RAMCACHE
|
|
||||||
tagcache_is_in_ram() ? true :
|
|
||||||
#endif
|
|
||||||
storage_disk_is_active()
|
|
||||||
#else
|
|
||||||
true
|
|
||||||
#endif
|
|
||||||
, 0, 0, 0);
|
|
||||||
|
|
||||||
if (c->currtable == TABLE_ALLSUBENTRIES)
|
if (c->currtable == TABLE_ALLSUBENTRIES)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue