On 1/23/19 4:46 PM, Cole Robinson wrote:
This is a follow on from a discussion with Andrea here: https://www.redhat.com/archives/libvir-list/2019-January/msg00750.html
Most VIR_ENUM_IMPLS look like this:
VIR_ENUM_IMPL(XXX, XXX, "value1", "value2");
But he and I find this format nicer:
VIR_ENUM_IMPL(XXX, XXX, "value1", "value2", );
Me too :)
The latter format means new enum additions only trigger 1 line of diff, vs 2 for the current common method.
Since this patch series is already touching a lot of VIR_ENUM_IMPL endings, now is as good a time as any to widely switch to that format if others agree.
Do we have a syntax-check rule to enforce the new style? -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org