[SSSD] [PATCH] FAIL_OVER: Fix warning value computed is not used

Lukas Slebodnik lslebodn at redhat.com
Tue Nov 10 14:02:24 UTC 2015


On (10/11/15 14:55), 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 is a macro which return itself.
Without changing anything.

It is basicly noop.
  int a = 1;
  a = a;

LS


More information about the sssd-devel mailing list