Commit f6332edf authored by Jean-Paul Saman's avatar Jean-Paul Saman

Remove useless comparisson (n >= 0) was always true (n is unsigned int)

parent b20a4ac4
...@@ -73,7 +73,7 @@ void Popup::addSeparator( int pos ) ...@@ -73,7 +73,7 @@ void Popup::addSeparator( int pos )
void Popup::handleEvent( const EvtMenu &rEvent ) void Popup::handleEvent( const EvtMenu &rEvent )
{ {
unsigned int n = m_pOsPopup->getPosFromId( rEvent.getItemId() ); unsigned int n = m_pOsPopup->getPosFromId( rEvent.getItemId() );
if( n >= 0 && n < m_actions.size() && m_actions[n] ) if( (n < m_actions.size()) && m_actions[n] )
{ {
m_actions[n]->execute(); m_actions[n]->execute();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment