On Mon, Apr 07, 2014 at 06:13:53PM +0200, Sumit Bose wrote:
> On Mon, Apr 07, 2014 at 02:22:04PM +0200, Pavel Reichl wrote:
> > Hello,
> > 
> > I noticed these two warnings in clang.
> > 
> > It would be great if the 2nd patch could be checked by Sumit to make
> > sure that the return value wasn't ignored on purpose.
> 
> yes, I would prefer to ignore errors here. There might be various cases
> were we are not able to resolve a single SID but still can proceed with
> the others.
> 
> The first patch looks good, I'll run some tests and will give my results
> later.

ACK to the first patch.

bye,
Sumit

> 
> bye,
> Sumit
> 
> > 
> > Thanks,
> > 
> > Pavel Reichl
> > 
> _______________________________________________
> sssd-devel mailing list
> sssd-devel@lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to