aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2015-08-11 21:54:29 +0200
committerMax Kellermann <max@duempel.org>2015-08-11 21:54:29 +0200
commit993df0fd289d4426c633fd6e6d12bffe6061599d (patch)
treee0f9d7b7477568f3b7dacf36b5f24d54ad9e73b8 /src
parent4294fdb0b26ec6af77d1fc671e1b88941af10bd2 (diff)
downloadmpd-993df0fd289d4426c633fd6e6d12bffe6061599d.tar.gz
mpd-993df0fd289d4426c633fd6e6d12bffe6061599d.tar.xz
mpd-993df0fd289d4426c633fd6e6d12bffe6061599d.zip
command/{Queue,Other}: eliminate local "bool" variable
Diffstat (limited to 'src')
-rw-r--r--src/command/OtherCommands.cxx5
-rw-r--r--src/command/QueueCommands.cxx4
-rw-r--r--src/command/StickerCommands.cxx10
3 files changed, 6 insertions, 13 deletions
diff --git a/src/command/OtherCommands.cxx b/src/command/OtherCommands.cxx
index 9ca7567b9..6ef9b0bd7 100644
--- a/src/command/OtherCommands.cxx
+++ b/src/command/OtherCommands.cxx
@@ -308,8 +308,6 @@ CommandResult
handle_setvol(Client &client, ConstBuffer<const char *> args)
{
unsigned level;
- bool success;
-
if (!check_unsigned(client, &level, args.front()))
return CommandResult::ERROR;
@@ -318,8 +316,7 @@ handle_setvol(Client &client, ConstBuffer<const char *> args)
return CommandResult::ERROR;
}
- success = volume_level_change(client.partition.outputs, level);
- if (!success) {
+ if (!volume_level_change(client.partition.outputs, level)) {
command_error(client, ACK_ERROR_SYSTEM,
"problems setting volume");
return CommandResult::ERROR;
diff --git a/src/command/QueueCommands.cxx b/src/command/QueueCommands.cxx
index d461170fd..561ef1027 100644
--- a/src/command/QueueCommands.cxx
+++ b/src/command/QueueCommands.cxx
@@ -249,13 +249,11 @@ CommandResult
handle_playlistinfo(Client &client, ConstBuffer<const char *> args)
{
unsigned start = 0, end = std::numeric_limits<unsigned>::max();
- bool ret;
if (args.size == 1 && !check_range(client, &start, &end, args.front()))
return CommandResult::ERROR;
- ret = playlist_print_info(client, client.playlist, start, end);
- if (!ret)
+ if (!playlist_print_info(client, client.playlist, start, end))
return print_playlist_result(client,
PlaylistResult::BAD_RANGE);
diff --git a/src/command/StickerCommands.cxx b/src/command/StickerCommands.cxx
index 8979d53a4..9dda3c65d 100644
--- a/src/command/StickerCommands.cxx
+++ b/src/command/StickerCommands.cxx
@@ -166,17 +166,15 @@ handle_sticker_song(Client &client, ConstBuffer<const char *> args)
}
}
- bool success;
struct sticker_song_find_data data = {
client,
args[3],
};
- success = sticker_song_find(*db, base_uri, data.name,
- op, value,
- sticker_song_find_print_cb, &data,
- error);
- if (!success) {
+ if (!sticker_song_find(*db, base_uri, data.name,
+ op, value,
+ sticker_song_find_print_cb, &data,
+ error)) {
if (error.IsDefined())
return print_error(client, error);