[vlc-devel] RE : commit: skins2: skins2 interface cannot be usedif no playlist is available (Erwan Tulou )

brezhoneg1 brezhoneg1 at yahoo.fr
Sun Sep 20 15:08:44 CEST 2009


> >  modules/gui/skins2/src/skin_main.cpp |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> >
> > diff --git a/modules/gui/skins2/src/skin_main.cpp
> b/modules/gui/skins2/src/skin_main.cpp
> > index 4c6bbbc..6aa0e47 100644
> > --- a/modules/gui/skins2/src/skin_main.cpp
> > +++ b/modules/gui/skins2/src/skin_main.cpp
> > @@ -106,6 +106,11 @@ static int Open( vlc_object_t *p_this )
> >
> >      p_intf->p_sys->p_input = NULL;
> >      p_intf->p_sys->p_playlist = pl_Hold( p_intf );
> > +    if( !p_intf->p_sys->p_playlist )
> > +    {
> > +        free( p_intf->p_sys );
> > +        return VLC_EGENERIC;
> > +    }
>  In fact, you could even abort/assert here. VLC core will not continue
> if the playlist creation have failed.
>  By design, pl_Hold cannot failed while an interface (launched by the
> core)
> is alive (for 1.x at least).
> 

Okay, I didn't know. Thanks for stating it.

That reminds me of reading somewhere something about making the playlist
optional (It can be a useless overhead for users only working with
libvlc or vlm). Any news ?

Rgds
Erwan10






More information about the vlc-devel mailing list