diff options
author | Max Kellermann <max@duempel.org> | 2015-08-11 21:54:29 +0200 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2015-08-11 21:54:29 +0200 |
commit | 993df0fd289d4426c633fd6e6d12bffe6061599d (patch) | |
tree | e0f9d7b7477568f3b7dacf36b5f24d54ad9e73b8 | |
parent | 4294fdb0b26ec6af77d1fc671e1b88941af10bd2 (diff) | |
download | mpd-993df0fd289d4426c633fd6e6d12bffe6061599d.tar.gz mpd-993df0fd289d4426c633fd6e6d12bffe6061599d.tar.xz mpd-993df0fd289d4426c633fd6e6d12bffe6061599d.zip |
command/{Queue,Other}: eliminate local "bool" variable
-rw-r--r-- | src/command/OtherCommands.cxx | 5 | ||||
-rw-r--r-- | src/command/QueueCommands.cxx | 4 | ||||
-rw-r--r-- | src/command/StickerCommands.cxx | 10 |
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); |