Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Use qBound() instead of multiple ifs.

Browse Source 
(cherry picked from commit 5fc6804326fa34cf2402b95c500f1fa25164cfae)
This commit is contained in:
Andreas 2013-08-20 11:07:58 +02:00
parent 2d921dbe47
commit 23aff7c890
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/networkremote/incomingdataparser.cpp
View File

@ -300,11 +300,7 @@ void IncomingDataParser::ClosePlaylist(const pb::remote::Message &msg) {
} }
void IncomingDataParser::RateSong(const pb::remote::Message &msg) { void IncomingDataParser::RateSong(const pb::remote::Message &msg) {
int rating = msg.request_rate_song().rating(); int rating = qBound(0, msg.request_rate_song().rating(), 5);
// Rating is from 0 to 5
if (rating > 5) rating = 5;
if (rating < 0) rating = 0;
emit RateCurrentSong(rating); emit RateCurrentSong(rating);
} }