[vlc-devel] [PATCH]Fix for the shoutcast bug in Qt playlist
Ilkka Ollakka
ilkka.ollakka+vlc at oamk.fi
Tue Aug 12 09:00:57 CEST 2008
On ti 12. elokuuta 2008 01:54:34, Pierre d'Herbemont wrote:
>
> On Aug 11, 2008, at 9:22 PM, Geoffroy Couprie wrote:
>
> > I removed the --playlist-tree, it is now always taking effect. That
> > was the easier way to workaround #1668. AFAIK it won't harm the other
> > interfaces, the b_tree option was a Qt only thing.
> > Please review, comment, and push, or slap me in the face if that's a
> > bad patch :)
>
> I don't use Qt so I may be biaised. It makes sense to stop using that
> b_tree option. This introduces some overhead in the playlist code, and
> is mostly the reason why it is hard to maintain.
>
> Yet doing so would probably break displaying the Qt4 interfaces as
> flat view. I don't know if someone cares.
I care, I usually view playlist as flat view (and treeview don't expand
all open on default, thats the reason actually that I don't use it).
--
Ilkka Ollakka
What is a magician but a practising theorist?
-- Obi-Wan Kenobi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20080812/6a74dea3/attachment.sig>
More information about the vlc-devel
mailing list