10 May
2010
10 May
'10
4:33 p.m.
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? -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org