Re: [libvirt] [PATCH 11/11] maint: enforce comma style usage

On 11/20/2013 06:26 AM, Osier Yang wrote: Sure; I'll squash this in. diff --git i/docs/hacking.html.in w/docs/hacking.html.in index cc76997..0febee2 100644 --- i/docs/hacking.html.in +++ w/docs/hacking.html.in @@ -417,10 +418,15 @@ When declaring an enum or using a struct initializer that occupies more than one line, use a trailing comma. That way, future edits to extend the list only have to add a line, rather - than modify an existing line to add the intermediate comma. - However, this is harder to enforce, so you will find - counterexamples in existing code. Additionally, any sentinel - enumerator value with a name ending in _LAST is exempt. + than modify an existing line to add the intermediate comma. Any + sentinel enumerator value with a name ending in _LAST is exempt, + since you would extend such an enum before the _LAST element. + Another reason to favor trailing commas is that it requires less + effort to produce via code generators. Note that the syntax + checker is unable to enforce a style of trailing commas, so + there are counterexamples in existing code which do not use it; + also, while C99 allows trailing commas, remember that JSON and + XDR do not. </p> <pre> enum { Thanks for the quick review. I'll push the series shortly. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org