<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 25 Jul 2019 at 4:44 PM, Alexandre Janniaux <<a href="mailto:ajanni@videolabs.io">ajanni@videolabs.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Actually I was a bit to fast in answering, it currently displays nothing in<br>
vlc -H but could in the future (only add_module does that currently) and<br>
could also be presented just like control interfaces in the Qt interfaces.<br>
<br>
It also provides better insight when reading the code because you'll know<br>
the capability supported for this variable.<br>
<br>
You shouldn't have to introduce this kind of documentation yourself, as the<br>
module system is dynamic and you never know when compiling which modules will<br>
be available at runtime.<br>
<br>
Thank you for your changes :)</blockquote><div dir="auto"><br></div><div dir="auto">Ok, sounds good to me :-)</div><div dir="auto"><br></div><div dir="auto">Thanks for the explanation.</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Regards,<br>
<br>
--<br>
Alexandre Janniaux<br>
VideoLabs<br>
<br>
On Thu, Jul 25, 2019 at 01:32:54PM +0700, Roland Bewick wrote:<br>
><br>
> On 24/07/2019 11:29 PM, Alexandre Janniaux wrote:<br>
> > The default value should probably be NULL like "keystore", "clock-source",<br>
> > pidfile,... btw, to stay consistent. :)<br>
> Sounds good.<br>
> ><br>
> > --<br>
> > Alexandre Janniaux<br>
> > VideoLabs<br>
> ><br>
> > On Wed, Jul 24, 2019 at 06:11:16PM +0200, Alexandre Janniaux wrote:<br>
> > > Hi,<br>
> > ><br>
> > > This would probably be a good idea to use add_module_list instead of<br>
> > > add_string, as `vlc -H` will provide the available module that you can<br>
> > > put there and you probably want to be capable of running multiple<br>
> > > service advertisement like the variable name states.<br>
> > > If so, you'll need to use config_ChainParse to extract the different<br>
> > > module and their configurations.<br>
><br>
> I've updated to use add_module_list and I just get the<br>
> SERVICE_ADVERTISEMENT_TEXT and SERVICE_ADVERTISEMENT_LONGTEXT:<br>
><br>
>       --services-advertisement=<string><br>
>                                  Services advertisement modules<br>
>           Specifies the services advertisement modules to preload, separated<br>
> by colons. Typical value is "mdns".<br>
><br>
><br>
> Do you mean it should show something like the below?<br>
><br>
> --services-advertisement={mdns}<br>
><br>
> > ><br>
> > > Besides, other defines aren't using abbreviations so maybe SA_TEXT could<br>
> > > be a more explicit name ?<br>
> Renamed SERVICE_ADVERTISEMENT_*<br>
> > ><br>
> > > Regards,<br>
> > ><br>
> > > --<br>
> > > Alexandre Janniaux<br>
> > > VideoLabs<br>
> > ><br>
> > > On Wed, Jul 24, 2019 at 10:43:45PM +0700, Roland Bewick wrote:<br>
> > > > ---<br>
> > > >   src/libvlc-module.c | 6 ++++++<br>
> > > >   1 file changed, 6 insertions(+)<br>
> > > ><br>
> > > > diff --git a/src/libvlc-module.c b/src/libvlc-module.c<br>
> > > > index f5de78bcae..97075cbab6 100644<br>
> > > > --- a/src/libvlc-module.c<br>
> > > > +++ b/src/libvlc-module.c<br>
> > > > @@ -1232,6 +1232,11 @@ static const char *const psz_recursive_list_text[] = {<br>
> > > >        "Specifies the services discovery modules to preload, separated by " \<br>
> > > >        "colons. Typical value is \"sap\"." )<br>
> > > ><br>
> > > > +#define SA_TEXT N_( "Services advertisement modules")<br>
> > > > +#define SA_LONGTEXT N_( \<br>
> > > > +     "Specifies the services advertisement modules to preload, separated by " \<br>
> > > > +     "colons. Typical value is \"mdns\"." )<br>
> > > > +<br>
> > > >   #define RANDOM_TEXT N_("Play files randomly forever")<br>
> > > >   #define RANDOM_LONGTEXT N_( \<br>
> > > >       "VLC will randomly play files in the playlist until interrupted.")<br>
> > > > @@ -2125,6 +2130,7 @@ vlc_module_begin ()<br>
> > > >       add_category_hint(N_("Miscellaneous"), MISC_CAT_LONGTEXT)<br>
> > > >       add_module("vod-server", "vod server", NULL,<br>
> > > >                  VOD_SERVER_TEXT, VOD_SERVER_LONGTEXT)<br>
> > > > +    add_string( "services-advertisement", "", SA_TEXT, SA_LONGTEXT, true )<br>
> > > ><br>
> > > >       set_section( N_("Plugins" ), NULL )<br>
> > > >   #ifdef HAVE_DYNAMIC_PLUGINS<br>
> > > > --<br>
> > > > 2.11.0<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><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><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><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><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></div>