On Tue, 2016-03-01 at 13:35 +0100, Pino Toscano wrote:
 On Tuesday 01 March 2016 11:17:19 Cédric Bosdonnat wrote: 
[...]
 > +      if (asprintf (&p, "%s:/:subvol=%s", device,
volume) == -1) {
 > +          perror ("asprintf");
 > +          exit (EXIT_FAILURE);
 > +      }
 > +    } else {
 > +      p = guestfs_canonical_device_name (g, fses[i]);
 > +    }
 
 Usually there are no curly brackets for blocks of a single
 instruction. 
Even if there is a non-single line block in the same if (or else) part?
--
Cedric