[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCHv2] storage_encryption: silence clang warning



Eric Blake wrote:

> On 05/10/2010 02:32 PM, Jim Meyering wrote:
>>
>> Using the cast is the lesser evil ;)
>>
>>>>
>>>> Of course, avoiding casts is good, too, but IMHO, not if
>>>> it makes us obfuscate (even ever so slightly) the types we use.
>>>
>>> Well, the real point of this patch was to silence a compiler warning
>>> (not that we'd ever planning on passing an indent > 2G).  Given Dave's
>>> ACK, I went ahead and applied v2 as proposed, even if it does slightly
>>> obfuscate the usage.
>>
>> Warning removal is important.
>> But it can end up being counterproductive if not done carefully.
>
> Should I go ahead and revert v2, going back to v1 which localized the
> change to just one file?

This variable is not important.
It's the principle, as I tried to explain:

  > I'm not lobbying for further change, but merely
  > making sure you see where I'm coming from.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]