[vlc-devel] commit: qt4: remove b_is_node and use childCount() > 0 instead in playlist_model (Ilkka Ollakka )

[Jakob Leben]

On Mon, Aug 17, 2009 at 10:18 PM, Ilkka Ollakka
<ilkka.ollakka+vlc at oamk.fi<ilkka.ollakka%2Bvlc at oamk.fi>
> wrote:

>  Is there any real cases where it matters? eg why would there be
>  significant nodes in playlist that has to show as nodes even if it's
>  empty ?


This triggers a bigger issue: why is it possible to delete children of nodes
when it has no sense for a node to be empty?
But the point right now is that it is possible to empty a node and maybe you
want to fill it later again. Refilling it will not be possible as the node
will not be Qt::ItemIsDropEnabled anymore.

If you want to keep current implementation then we might just as well
disable deleting children of nodes. But I suggest we look forward to a day
when it will be possible to create arbitrary nodes with preffered names and
add/remove items to them at will.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090817/5712b249/attachment.html>