driver: play() and pause() shouldn't return a boolean

This value is never used in the audio layer so let's stop returning it
from the driver.

Signed-off-by: Anna Schumaker <Anna@OcarinaProject.net>

gui/gst.cpp

@@ -42,8 +42,8 @@ public:
 		on_track_loaded(track);
 	}
 
-	bool play()  { return gst_change_state(GST_STATE_PLAYING); }
+	void play()  { gst_change_state(GST_STATE_PLAYING); }
-	bool pause() { return gst_change_state(GST_STATE_PAUSED);  }
+	void pause() { gst_change_state(GST_STATE_PAUSED);  }
 
 	bool is_playing()
 	{

include/core/driver.h

@@ -34,17 +34,13 @@ public:
 
 	/**
 	 * Called to begin playback on the currently loaded track.
-	 *
-	 * @return True if playback state was changed successfully, false otherwise.
 	 */
-	virtual bool play() = 0;
+	virtual void play() = 0;
 
 	/**
 	 * Called to pause playback on the currently loaded track.
-	 *
-	 * @return True if playback state was changed successfully, false otherwise.
 	 */
-	virtual bool pause() = 0;
+	virtual void pause() = 0;
 
 	/**
 	 * Called to check if the audio library is currently playing a track.

tests/core/audio.cpp

@@ -28,17 +28,8 @@ public:
 		cur_pos  = 0;
 	}
 
-	bool play()
+	void play()  { playing = true;  }
-	{
+	void pause() { playing = false; }
-		playing = true;
-		return true;
-	}
-
-	bool pause()
-	{
-		playing = false;
-		return true;
-	}
 
 	bool is_playing() { return playing; }