[vlc-devel] Re: [patch] mediacontrol core.c new function

Olivier Aubert oaubert at lisi.univ-lyon1.fr
Mon Dec 12 13:09:03 CET 2005


>  I'm sending this one in order to get a review on this code and to
> know if I can generally proceed in this way:

> +void
> +mediacontrol_playlist_next_item( mediacontrol_Instance *self,
> +                                 mediacontrol_Exception *exception )
> +{
> +    exception=mediacontrol_exception_init( exception );
> +    if ( !self->p_playlist )
> +    {
> +        RAISE( mediacontrol_InternalException, "No playlist" );
> +        return;
> +    }
> +    playlist_Next( self->p_playlist );
> +}
Looks fine. Maybe we should remove the _item (from playlist_add_item
also), it looks redundant anyway. There are other suggestions on the
wiki.

Olivier


-- 
Olivier Aubert <oaubert at lisi.univ-lyon1.fr>

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html



More information about the vlc-devel mailing list