From 97e2f1e5885ae2e58249c9b81050d7e67904c35b Mon Sep 17 00:00:00 2001 From: "Uwe L. Korn" Date: Thu, 18 Sep 2014 23:38:06 +0100 Subject: [PATCH] No need to call QString::number on an parameter to QString::arg() --- src/libtomahawk/playlist/PlayableModel.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/libtomahawk/playlist/PlayableModel.cpp b/src/libtomahawk/playlist/PlayableModel.cpp index 1a53bcbe8..9e2e4a91c 100644 --- a/src/libtomahawk/playlist/PlayableModel.cpp +++ b/src/libtomahawk/playlist/PlayableModel.cpp @@ -242,14 +242,14 @@ PlayableModel::queryData( const query_ptr& query, int column, int role ) const case AlbumPos: { - QString tPos; - if ( query->track()->albumpos() != 0 ) + const uint tPos = query->track()->albumpos(); + if ( tPos != 0 ) { - tPos = QString::number( query->track()->albumpos() ); + const uint discnumber = query->track()->discnumber(); if ( query->track()->discnumber() == 0 ) - return tPos; + return QString::number( tPos ); else - return QString( "%1.%2" ).arg( QString::number( query->track()->discnumber() ) ) + return QString( "%1.%2" ).arg( discnumber ) .arg( tPos ); } }