On Thu, May 19, 2016 at 08:08:39AM -0400, Cole Robinson wrote: > On 05/18/2016 02:36 AM, Nishith Shah wrote: > > + virDomainDefSetMemoryTotal(dom, mem * 1024); > > + dom->mem.cur_balloon = mem * 1024; > > + > > + return 0; > > + > > + error: > > + return -1; > > +} > > + > > Typically when we use the 'goto error' pattern, what we do is: > > int ret = -1; > > if (condition) > goto error; > > ret = 0; > error: > return ret; > > So there's only one 'return' When the path is taken both on error and success, 'cleanup' is the preferred label name: http://libvirt.org/hacking.html#goto Jan -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list