queue/PlaylistControl: use PlaylistError::NotPlaying()

aca08155 · Max Kellermann · a71efacd · aca08155
Commit aca08155 authored Sep 05, 2016 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 10 deletions

PlaylistControl.cxx src/queue/PlaylistControl.cxx +4 -10

No files found.
--- a/src/queue/PlaylistControl.cxx
+++ b/src/queue/PlaylistControl.cxx
@@ -121,11 +121,8 @@ playlist::PlayId(PlayerControl &pc, int id, Error &error)
 bool
 playlist::PlayNext(PlayerControl &pc, Error &error)
 {
-	if (!playing) {
-		error.Set(playlist_domain, int(PlaylistResult::NOT_PLAYING),
-			  "Not playing");
-		return false;
-	}
+	if (!playing)
+		throw PlaylistError::NotPlaying();

 	assert(!queue.IsEmpty());
 	assert(queue.IsValidOrder(current));
@@ -173,11 +170,8 @@ playlist::PlayNext(PlayerControl &pc, Error &error)
 bool
 playlist::PlayPrevious(PlayerControl &pc, Error &error)
 {
-	if (!playing) {
-		error.Set(playlist_domain, int(PlaylistResult::NOT_PLAYING),
-			  "Not playing");
-		return false;
-	}
+	if (!playing)
+		throw PlaylistError::NotPlaying();

 	assert(!queue.IsEmpty());