[vlc-devel] [PATCH 2/2] fix for #15713, checked for i_type, if i_type is ITEM_TYPE_DIRECTORY notification "Now Playing" is not displayed

[Avani Sharma]

Yeah sure jb :). I would like to explore this.

Avani

On Fri, Mar 18, 2016 at 5:02 AM, Jean-Baptiste Kempf <jb at videolan.org>
wrote:

> On 18 Mar, Avani Sharma wrote :
> > ---
> >  modules/notify/notify.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/modules/notify/notify.c b/modules/notify/notify.c
> > index adc6de2..1fe1eb3 100644
> > --- a/modules/notify/notify.c
> > +++ b/modules/notify/notify.c
> > @@ -170,10 +170,16 @@ static int ItemChange( vlc_object_t *p_this, const
> > char *psz_var,
> >      if( !p_input )
> >          return VLC_SUCCESS;
> >
> > +
>
> Unneeded.
>
> >      /* Playing something ... */
> >      input_item_t *p_input_item = input_GetItem( p_input );
> >      psz_title = input_item_GetTitleFbName( p_input_item );
> >
> > +    /* Checking for click on directories */
> > +    if(p_input_item->i_type == ITEM_TYPE_DIRECTORY){
> > +        return VLC_SUCCESS;
> > +    }
>
> Way better! Maybe there are other ITEM_TYPE we should ignore, don'tyou
> thinkg?
>
> -
> With my kindest regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/ - +33 672 704 734
> Sent from my Electronic Device
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20160318/f2719623/attachment.html>