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

[Rémi Denis-Courmont]

Le lundi 15 juin 2009 10:20:49 git version control, vous avez écrit :
> vlc | branch: master | Rémi Duraffort <ivoire at videolan.org> | Mon Jun 15
> 08:49:46 2009 +0200| [e4a867371a688d8fc4bcfc45c9d61d4386100b05] |
> committer: Rémi Duraffort
>
> Use add_password instead of add_string when applicable.
>
> > http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=e4a867371a688d8fc4
> >bcfc45c9d61d4386100b05
>
> ---
>
>  modules/access/dvb/access.c  |    4 ++--
>  modules/access/ftp.c         |    4 ++--
>  modules/access/smb.c         |    4 ++--
>  modules/access_output/http.c |    4 ++--
>  modules/demux/live555.cpp    |    4 ++--
>  5 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/modules/access/dvb/access.c b/modules/access/dvb/access.c
> index df8cd1c..4726aab 100644
> --- a/modules/access/dvb/access.c
> +++ b/modules/access/dvb/access.c
> @@ -250,8 +250,8 @@ vlc_module_begin ()
>                  true )
>      add_string( "dvb-http-user", NULL, NULL, USER_TEXT, USER_LONGTEXT,
>                  true )
> -    add_string( "dvb-http-password", NULL, NULL, PASSWORD_TEXT,
> -                PASSWORD_LONGTEXT, true )
> +    add_password( "dvb-http-password", NULL, NULL, PASSWORD_TEXT,
> +                  PASSWORD_LONGTEXT, true )
>      add_string( "dvb-http-acl", NULL, NULL, ACL_TEXT, ACL_LONGTEXT,
>                  true )
>      add_string( "dvb-http-intf-cert", NULL, NULL, CERT_TEXT,
> 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...
-- 
Rémi Denis-Courmont
http://www.remlab.net/