<div dir="ltr"><div>Before, the playlist was created on the first call to pl_Get, this is not the case anymore.</div><div>I offer to instantiate the playlist (non-populated) and then the system default interface.</div><div>With this patch the playlist is still populated after the D-Bus code is evaluated, since afaik</div><div>there is only two ways to populate it, either through an interface, or though the command line.</div><div>And the latter is done a bit further in the same function.<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 27, 2018 at 7:40 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le tiistaina 27. marraskuuta 2018, 14.28.56 EET Victorien Le Couviour--Tuffet <br>
a écrit :<br>
> The playlist needs to be created before spawning the default system<br>
> interface since it might use it (to setup playlist / player listeners<br>
> for instance).<br>
<br>
No. The D-Bus code must be evaluated before the playlist is populated.<br>
<br>
<br>
> ---<br>
>  src/libvlc.c | 6 +++---<br>
>  1 file changed, 3 insertions(+), 3 deletions(-)<br>
> <br>
> diff --git a/src/libvlc.c b/src/libvlc.c<br>
> index 8061171cae..cf35bcfed3 100644<br>
> --- a/src/libvlc.c<br>
> +++ b/src/libvlc.c<br>
> @@ -322,9 +322,6 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int<br>
> i_argc, var_Create( p_libvlc, "app-version", VLC_VAR_STRING );<br>
>      var_SetString( p_libvlc, "app-version", PACKAGE_VERSION );<br>
> <br>
> -    /* System specific configuration */<br>
> -    system_Configure( p_libvlc, i_argc - vlc_optind, ppsz_argv + vlc_optind<br>
> ); -<br>
>  #ifdef ENABLE_VLM<br>
>      /* Initialize VLM if vlm-conf is specified */<br>
>      psz_parser = var_InheritString( p_libvlc, "vlm-conf" );<br>
> @@ -343,6 +340,9 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int<br>
> i_argc,<br>
> <br>
>      PlaylistConfigureFromVariables(priv->main_playlist,<br>
> VLC_OBJECT(p_libvlc));<br>
> <br>
> +    /* System specific configuration */<br>
> +    system_Configure( p_libvlc, i_argc - vlc_optind, ppsz_argv + vlc_optind<br>
> ); +<br>
>      /*<br>
>       * Load background interfaces<br>
>       */<br>
<br>
<br>
-- <br>
雷米‧德尼-库尔蒙<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
<br>
<br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>