Skip to content

M
mpd
Commit a71efacd authored by Max Kellermann's avatar Max Kellermann
Browse files
Options

queue/PlaylistControl: fix return value in PlayNext(), PlayPrevious()

Failure must be indicated by returning false.
parent 871063da
Show whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
src/queue/PlaylistControl.cxx
View file @ a71efacd
...@@ -124,7 +124,7 @@ playlist::PlayNext(PlayerControl &pc, Error &error) ...@@ -124,7 +124,7 @@ playlist::PlayNext(PlayerControl &pc, Error &error)
if (!playing) { if (!playing) {
error.Set(playlist_domain, int(PlaylistResult::NOT_PLAYING), error.Set(playlist_domain, int(PlaylistResult::NOT_PLAYING),
"Not playing"); "Not playing");
return true; return false;
} }
assert(!queue.IsEmpty()); assert(!queue.IsEmpty());
...@@ -176,7 +176,7 @@ playlist::PlayPrevious(PlayerControl &pc, Error &error) ...@@ -176,7 +176,7 @@ playlist::PlayPrevious(PlayerControl &pc, Error &error)
if (!playing) { if (!playing) {
error.Set(playlist_domain, int(PlaylistResult::NOT_PLAYING), error.Set(playlist_domain, int(PlaylistResult::NOT_PLAYING),
"Not playing"); "Not playing");
return true; return false;
} }
assert(!queue.IsEmpty()); assert(!queue.IsEmpty());
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment