Commit a7f80fa2 authored by Erwan Tulou's avatar Erwan Tulou

skins2: fix wrong check (cosmetics)

For clarity, i_children can be -1 (at init) or 0 for no child. Yet, there was
no bug, since buildNode was just a noop for -1.

pointed out on the forum
parent 53391ce6
...@@ -248,7 +248,7 @@ void Playtree::buildNode( playlist_item_t *pNode, VarTree &rTree ) ...@@ -248,7 +248,7 @@ void Playtree::buildNode( playlist_item_t *pNode, VarTree &rTree )
playlist_CurrentPlayingItem(m_pPlaylist) == pNode->pp_children[i], playlist_CurrentPlayingItem(m_pPlaylist) == pNode->pp_children[i],
false, pNode->pp_children[i]->i_flags & PLAYLIST_RO_FLAG, false, pNode->pp_children[i]->i_flags & PLAYLIST_RO_FLAG,
pNode->pp_children[i] ); pNode->pp_children[i] );
if( pNode->pp_children[i]->i_children ) if( pNode->pp_children[i]->i_children > 0 )
{ {
buildNode( pNode->pp_children[i], rTree.back() ); buildNode( pNode->pp_children[i], rTree.back() );
} }
......
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