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

Re: [libvirt] [PATCH 00/15] Cleanup some curly braces



On 11/06/2014 05:38 PM, Martin Kletzander wrote:
> There were unnecessary curly braces in few places (and the whole esx
> driver), so this series proposes fixing them.  As this couldn't be
> done automatically, there is no syntax-check.  We have to keep trying
> to eliminate that ;)

I'm a little bit worried that without a syntax check, this is just code
churn and a conflict magnet that will make it harder to backport
patches.  I'm not entirely opposed to the idea, but I'm also thinking we
should not be applying it without discussion.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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