[SSSD] [PATCHES] check return value
Sumit Bose
sbose at redhat.com
Tue Apr 8 11:00:57 UTC 2014
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 at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
More information about the sssd-devel
mailing list