Playlist*, Queue: use GetDatabase() overload without Error
Don't use IgnoreError() when there's an overload that does not try to give us one.
This commit is contained in:
parent
fc05768374
commit
3f899f83ff
@ -115,7 +115,7 @@ playlist::AppendURI(struct player_control &pc,
|
||||
if (uri_has_scheme(uri)) {
|
||||
song = Song::NewRemote(uri);
|
||||
} else {
|
||||
db = GetDatabase(IgnoreError());
|
||||
db = GetDatabase();
|
||||
if (db == nullptr)
|
||||
return PlaylistResult::NO_SUCH_SONG;
|
||||
|
||||
|
@ -115,7 +115,7 @@ playlist_print_changes_position(Client &client,
|
||||
static bool
|
||||
PrintSongDetails(Client &client, const char *uri_utf8)
|
||||
{
|
||||
const Database *db = GetDatabase(IgnoreError());
|
||||
const Database *db = GetDatabase();
|
||||
if (db == nullptr)
|
||||
return false;
|
||||
|
||||
|
@ -104,7 +104,7 @@ playlist_check_load_song(const Song *song, const char *uri, bool secure)
|
||||
if (dest == nullptr)
|
||||
return nullptr;
|
||||
} else {
|
||||
const Database *db = GetDatabase(IgnoreError());
|
||||
const Database *db = GetDatabase();
|
||||
if (db == nullptr)
|
||||
return nullptr;
|
||||
|
||||
|
@ -114,7 +114,7 @@ queue_load_song(TextFile &file, const char *line, queue &queue)
|
||||
if (uri_has_scheme(uri)) {
|
||||
song = Song::NewRemote(uri);
|
||||
} else {
|
||||
db = GetDatabase(IgnoreError());
|
||||
db = GetDatabase();
|
||||
if (db == nullptr)
|
||||
return;
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user