PlayerControl: GetError() returns an Error, not a char*

src/PlayerCommands.cxx

@@ -32,8 +32,6 @@
 #include "AudioFormat.hxx"
 #include "ReplayGainConfig.hxx"
 
-#include <glib.h>
-
 #define COMMAND_STATUS_STATE            "state"
 #define COMMAND_STATUS_REPEAT           "repeat"
 #define COMMAND_STATUS_SINGLE           "single"
@@ -190,13 +188,11 @@ handle_status(Client *client,
 			      updateJobId);
 	}
 
-	char *error = client->player_control->GetErrorMessage();
+	Error error = client->player_control->LockGetError();
-	if (error != NULL) {
+	if (error.IsDefined())
 		client_printf(client,
 			      COMMAND_STATUS_ERROR ": %s\n",
-			      error);
+			      error.GetMessage());
-		g_free(error);
-	}
 
 	song = playlist.GetNextPosition();
 	if (song >= 0) {

src/PlayerControl.cxx

@@ -23,8 +23,6 @@
 #include "Song.hxx"
 #include "DecoderControl.hxx"
 
-#include <glib.h>
-
 #include <cmath>
 
 #include <assert.h>
@@ -202,17 +200,6 @@ player_control::ClearError()
 	Unlock();
 }
 
-char *
-player_control::GetErrorMessage() const
-{
-	Lock();
-	char *message = error_type != PlayerError::NONE
-		? g_strdup(error.GetMessage())
-		: NULL;
-	Unlock();
-	return message;
-}
-
 void
 player_control::EnqueueSong(Song *song)
 {

src/PlayerControl.hxx

@@ -325,14 +325,32 @@ public:
 	 */
 	void SetError(PlayerError type, Error &&error);
 
-	void ClearError();
+	/**
+	 * Checks whether an error has occurred, and if so, returns a
+	 * copy of the #Error object.
+	 *
+	 * Caller must lock the object.
+	 */
+	gcc_pure
+	Error GetError() const {
+		Error result;
+		if (error_type != PlayerError::NONE)
+			result.Set(error);
+		return result;
+	}
 
 	/**
-	 * Returns the human-readable message describing the last
+	 * Like GetError(), but locks and unlocks the object.
-	 * error during playback, NULL if no error occurred.  The
-	 * caller has to free the returned string.
 	 */
-	char *GetErrorMessage() const;
+	gcc_pure
+	Error LockGetError() const {
+		Lock();
+		Error result = GetError();
+		Unlock();
+		return result;
+	}
+
+	void ClearError();
 
 	PlayerError GetErrorType() const {
 		return error_type;