From 074713bb2917db5567a760966c0b3e95be451fac Mon Sep 17 00:00:00 2001 From: Thilo Graf Date: Mon, 2 Dec 2013 21:29:31 +0100 Subject: [PATCH] CAudioPlayerGui: assign value of variable 'step' with current value Variable 'step' is reassigned a value before the old one has been used. --- src/gui/audioplayer.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gui/audioplayer.cpp b/src/gui/audioplayer.cpp index af11e3cab..3180513c9 100644 --- a/src/gui/audioplayer.cpp +++ b/src/gui/audioplayer.cpp @@ -511,10 +511,9 @@ int CAudioPlayerGui::show() { if (m_show_playlist && !m_playlist.empty() ) { - int step = 0; int prevselected = m_selected; + int step = msg == CRCInput::RC_page_up ? m_listmaxshow : 1; - step = msg == CRCInput::RC_page_up ? m_listmaxshow : 1; m_selected -= step; if ((prevselected-step) < 0) m_selected = m_playlist.size()-1;