On 02/25/2016 02:04 PM, Jakub Hrozek wrote:
Hi,

attached is a simple clan-up patch.

Hi,
make_realm_upper_case also contains a check that returns the original string if delimiter is not found. Are we sure it is safe to remove this test?

Can you also rename get_uppercase_realm to something more general since it is not really realm specific? Maybe get_uppercase_string? Or sss_talloc_toupper?
_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to