Re: [libvirt] [PATCH 4/8] tests: Use virAsprintf() to build titles

[Andrea Bolognani]

On Wed, 2019-03-13 at 09:48 +0100, Peter Krempa wrote:
> On Thu, Mar 07, 2019 at 16:44:33 +0100, Andrea Bolognani wrote:
[...]
> >  #define DO_TEST(arch, name) \
> >      do { \
> > +        VIR_AUTOFREE(char *) title = NULL; \
> > +        VIR_AUTOFREE(char *) copyTitle = NULL; \
> > +        if (virAsprintf(&title, "%s (%s)", name, arch) < 0 || \
> > +            virAsprintf(&copyTitle, "copy %s (%s)", name, arch) < 0) { \
> > +            return -EXIT_FAILURE; \
> 
> Coding style. Single-line body.

There are multiple conditions with the same indentation, so per the
coding guidelines[1] the curly braces are required.

Honesly, we should really give clang-format or whatever similar tool
a serious go and just start enforcing that code needs to be filtered
through it before being merged. Having humans worry about this kind
of nonsense is such an utter waste of time.


[1] https://libvirt.org/hacking.html#curly_braces, third example.
-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list