[vlc-devel] vlc: svn commit r24474 (thresh)

[Rafaël Carré]

Le Tue, 22 Jan 2008 17:09:44 +0100,
"Olivier Teuliere" <ipkiss at gmail.com> a écrit :

> On Jan 22, 2008 4:41 PM, Rafaël Carré <funman at videolan.org> wrote:
> > > > 2/ It will be really hard to fix switching to skins2 from Qt4
> > > > (while providing the dialogs for the skins). I don't remember
> > > > the details exactly but basically there is things you can't
> > > > init twice in libQt4.
> > >
> > > that sucks, basically i cant switch from Qt4 to WX and then again,
> > > some dialogs not being fully destroyed? It works fine from vlc -I
> > > wxwidgets and then switch to QT4.
> >
> > Yeah that sucks, and that's why i'll remove this if someone doesn't
> > come with strong arguments.
> 
> Why not putting all the stuff which can't be initialized twice in a
> singleton, to be sure not to call them twice when switching
> interfaces?

Switching means unloading, and loading again another interface.
Such a thing could be set up, making all this thing even more hacky.

> I can think of one "uncommon" use of the interface switching: some
> features in the wxWidgets interface could not be accessed via the
> skins interface (controls of the extended GUI in particular), so
> having a way to switch from the skins to wx could be very convenient
> (but then it was removed, because of people coming from wx, who
> switched to skins, and switched back... *pouf*).
>
> So a possibility could be to allow switching from the skins interface
> to any other interface, and not the contrary (at least until the
> singleton mentioned above is put in place).

So you wouldn't be able to go back ? Don't run the skins2 in the first
place if you need some extra feature.

> I'm not sure this qualifies as a "strong argument" though :)

Without code, it's not that strong ;)

> Cheers,


-- 
Rafaël Carré
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20080122/52d3dc8a/attachment.sig>