[vlc-devel] [PATCH]Fix for the shoutcast bug in Qt playlist
Pierre d'Herbemont
pdherbemont at free.fr
Tue Aug 12 01:54:34 CEST 2008
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.
An other alternative is to revert
1f26dd3fc7a3a285a00e5e84db113ae4b496da5b and
722ab9f14ee31f8f0633cfc7a0b79401c47316fe. And to set the removed input-
>b_prefers_tree correctly in the shoutcast module.
Pierre.
More information about the vlc-devel
mailing list