[vlc-devel] commit: Use add_password instead of add_string when applicable. ( Rémi Duraffort )

[Rémi Duraffort]

> > Use add_password instead of add_string when applicable.
> >
> > CERT_LONGTEXT, diff --git a/modules/access/ftp.c b/modules/access/ftp.c
> > index b3dc2c9..b2d692d 100644
> > --- a/modules/access/ftp.c
> > +++ b/modules/access/ftp.c
> > @@ -79,8 +79,8 @@ vlc_module_begin ()
> >          change_safe()
> >      add_string( "ftp-user", "anonymous", NULL, USER_TEXT, USER_LONGTEXT,
> >                  false )
> > -    add_string( "ftp-pwd", "anonymous at example.com", NULL, PASS_TEXT,
> > -                PASS_LONGTEXT, false )
> > +    add_password( "ftp-pwd", "anonymous at example.com", NULL, PASS_TEXT,
> > +                  PASS_LONGTEXT, false )
> 
> IMHO, the default value does not make much sense anymore with this commit...
Yes you'r right. Maybe we can set the default value to NULL and give the
default value (anonymous at example.com) as an example in the long string ?

-- 
Rémi Duraffort | ivoire