Re: [libvirt] [PATCH v2 00/22] Remove unnecessary curly brackets around one-line bodies

On 11/13/2014 04:52 PM, John Ferlan wrote: I have not closely reviewed the series, and the perl in patch 22/22 is complex enough that I don't feel I have enough time to do it justice in a quick review today. I'll trust John's review - he did have a good point about multiline macro bodies needing do {} while (0) guards - maybe we add yet another syntax check for that? I'm guessing you wrote the series by applying patch 22 first, then fixing in pieces until things work; so if 22 is good and syntax-check passes, then all the other patches prove we silenced the syntax problems (but not necessarily that the transformations to silence the problems were correct, as was the case with macros in tools/). At any rate, this series will be a conflict magnet to other pending patches, so if we're going to take it, I don't want to hold it up for a long time. The fact that you have a syntax-check to enforce things is a huge improvement over the v1 proposal. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org