Commit 8da15a5a authored by Ludovic Fauvet's avatar Ludovic Fauvet

Returning -1 from unsigned method is most of the time a very bad idea.

parent cb73cf5d
......@@ -80,7 +80,9 @@ quint32 AbstractGraphicsMediaItem::trackNumber()
if ( graphicsTrack )
return graphicsTrack->trackNumber();
return -1;
//FIXME We should probably return a negative number here (using a signed type!)
qWarning( "Things can go wrong beyond that point!" );
return 0;
void AbstractGraphicsMediaItem::setTrack( GraphicsTrack* track )
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