[SSSD] [PATCH] FAIL_OVER: Fix warning value computed is not used
Jakub Hrozek
jhrozek at redhat.com
Tue Nov 10 14:05:33 UTC 2015
On Tue, Nov 10, 2015 at 02:55:43PM +0100, Pavel Reichl wrote:
>
>
> On 11/10/2015 02:39 PM, Jakub Hrozek wrote:
> >On Tue, Nov 10, 2015 at 09:11:25AM +0100, Lukas Slebodnik wrote:
> >>ehlo,
> >>
> >>another warning on rhel6.
> >
> >ACK
> >
> >>
> >>BTW different solution would be to cast to void.
> >
> >I prefer this solution
>
> Why? I think that changing value of input parameter is generally a discouraged practice, although in our code base it's common. (I'm just asking I'm *not* nacking the patch.)
It seems more readable that the parameter is changed. It's not really
input parameter per se, its refcount is increased.
More information about the sssd-devel
mailing list