[vlc-devel] [PATCH] m3u: Expose playlist title

[Rémi Denis-Courmont]

Le maanantaina 21. syyskuuta 2020, 18.42.48 EEST Hugo Beauzée-Luyssen a écrit 
:
> On Mon, Sep 21, 2020, at 4:59 PM, Rémi Denis-Courmont wrote:
> > Hi,
> > 
> > I saw that in Wikipedia yesterday but is it used anywhere?
> > 
> > Le 21 septembre 2020 16:54:17 GMT+03:00, "Hugo Beauzée-Luyssen"
> > <hugo at beauzee.fr> a écrit :  modules/demux/playlist/m3u.c | 6 ++++++
> > 
> > >  1 file changed, 6 insertions(+)
> > > 
> > > diff --git a/modules/demux/playlist/m3u.c b/modules/demux/playlist/m3u.c
> > > index af743b75da..2cf71fd8d8 100644
> > > --- a/modules/demux/playlist/m3u.c
> > > +++ b/modules/demux/playlist/m3u.c
> > > @@ -332,6 +332,12 @@ static int ReadDir( stream_t *p_demux,
> > > input_item_node_t *p_subitems )> > 
> > >                      meta.psz_album_art = pf_dup( psz_parse );
> > >                  
> > >                  }
> > >              
> > >              }
> > > 
> > > +            else if ( !strncasecmp( psz_parse, "PLAYLIST:",
> > > +                      sizeof( "PLAYLIST:" ) - 1 ) )
> > > +            {
> > > +                psz_parse += sizeof( "PLAYLIST:" ) - 1;
> > > +                input_item_SetTitle( p_demux->p_input_item, psz_parse
> > > );
> > > +            }
> > > 
> > >          }
> > >          else if( !strncasecmp( psz_parse, "RTSPtext",
> > >          sizeof("RTSPtext") -1 ) )
> > >          {
> 
> Hi,
> 
> I don't know how often it's used, but I had a complain on the medialib
> tracker about that:
> https://code.videolan.org/videolan/medialibrary/-/issues/266

I'll take that as a 'yes'.

LGTM

-- 
レミ・デニ-クールモン
http://www.remlab.net/