On 11/10/2015 03:05 PM, Jakub Hrozek wrote:
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.
OK, thanks both.
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel