On Tue, Oct 13, 2015 at 12:08:22PM +0200, Pavel Březina wrote:
> >I don't like overriding opt_name and opt_id here, the caller of
> >check_cache() does not expect that those values change (especially
> >because opt_name is declared const in the header). Additionally I'm not
> 
> The values won't change in the caller of check_cache.

But it's still really bad taste to change input parameters, can we have
a separate variable we assign to?
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to