decoder/Control: use ScopeLock

src/decoder/DecoderControl.cxx

@@ -90,7 +90,7 @@ DecoderControl::Start(DetachedSong *_song,
 void
 DecoderControl::Stop()
 {
-	Lock();
+	const ScopeLock protect(mutex);
 
 	if (command != DecoderCommand::NONE)
 		/* Attempt to cancel the current command.  If it's too
@@ -101,8 +101,6 @@ DecoderControl::Stop()
 
 	if (state != DecoderState::STOP && state != DecoderState::ERROR)
 		SynchronousCommandLocked(DecoderCommand::STOP);
-
-	Unlock();
 }
 
 bool

src/decoder/DecoderControl.hxx

@@ -213,10 +213,8 @@ struct DecoderControl {
 
 	gcc_pure
 	bool LockIsIdle() const {
-		Lock();
+		const ScopeLock protect(mutex);
-		bool result = IsIdle();
+		return IsIdle();
-		Unlock();
-		return result;
 	}
 
 	bool IsStarting() const {
@@ -225,10 +223,8 @@ struct DecoderControl {
 
 	gcc_pure
 	bool LockIsStarting() const {
-		Lock();
+		const ScopeLock protect(mutex);
-		bool result = IsStarting();
+		return IsStarting();
-		Unlock();
-		return result;
 	}
 
 	bool HasFailed() const {
@@ -239,10 +235,8 @@ struct DecoderControl {
 
 	gcc_pure
 	bool LockHasFailed() const {
-		Lock();
+		const ScopeLock protect(mutex);
-		bool result = HasFailed();
+		return HasFailed();
-		Unlock();
-		return result;
 	}
 
 	/**
@@ -267,10 +261,8 @@ struct DecoderControl {
 	 */
 	gcc_pure
 	Error LockGetError() const {
-		Lock();
+		const ScopeLock protect(mutex);
-		Error result = GetError();
+		return GetError();
-		Unlock();
-		return result;
 	}
 
 	/**
@@ -297,10 +289,8 @@ struct DecoderControl {
 
 	gcc_pure
 	bool LockIsCurrentSong(const DetachedSong &_song) const {
-		Lock();
+		const ScopeLock protect(mutex);
-		const bool result = IsCurrentSong(_song);
+		return IsCurrentSong(_song);
-		Unlock();
-		return result;
 	}
 
 private:
@@ -336,17 +326,15 @@ private:
 	 * object.
 	 */
 	void LockSynchronousCommand(DecoderCommand cmd) {
-		Lock();
+		const ScopeLock protect(mutex);
 		ClearError();
 		SynchronousCommandLocked(cmd);
-		Unlock();
 	}
 
 	void LockAsynchronousCommand(DecoderCommand cmd) {
-		Lock();
+		const ScopeLock protect(mutex);
 		command = cmd;
 		Signal();
-		Unlock();
 	}
 
 public: