On Mon, Sep 05, 2011 at 12:36:03PM +0200, Jakub Hrozek wrote:
On Mon, Sep 05, 2011 at 11:03:06AM +0200, Pavel Březina wrote:
>
https://fedorahosted.org/sssd/ticket/986
>
> [PATCH 1/2] sss_ldap_err2string() created
> [PATCH 2/2] ldap_err2string() changed to sss_ldap_err2string()
> +const char* sss_ldap_err2string(int err)
> +{
> + static const char *password_expired = "Password expired";
> +
> + switch (err) {
> + case LDAP_X_SSSD_PASSWORD_EXPIRED:
> + return password_expired;
Actually, do we want to treat these error strings as translatable? I
would think so, to treat them the same as ldap_err2string()
I had a closer look and libldap would only retun localized string if the
LDAP_LOCALIZE macro was on. Currently the only way to set it is to
compile with -DLDAP_LOCALIZE (there's no configure flag) and the macro
is not set in Fedora or RHEL by default. So I think we're safe with not
localizing.
By the way, is there a reason for defining the password_expired variable
instead of just returning the string constant?