Re: [libvirt] [PATCH v1 00/26] remove more unneeded labels

On Mon, Jan 06, 2020 at 06:57:24PM -0300, Daniel Henrique Barboza wrote: > Hi, > > This is a follow up of a cleanup I did last year [1], which was > more focused on 'cleanup' labels. In this one I covered all > labels that are simply a 'return' proxy call. > > This wasn't a 'purge all return labels' series. Some of them > are still around in cases where they help the code flow > (like Jano suggested in the review at [2], for example). > > I divided the series by module and by amount of changes made > to make it less daunting for reviews (although cases such as > domain_conf.c couldn't be avoided without splitting changes > in the same file in multiple patches). The maintainer is > welcome to squash the patches up as needed. Right, that's what I will do. Like I mentioned in 01/26, I can imagine more thorough Glib-ification of some of the patches, there are many candidates with a simple straightforward way to convert and thus drop even more labels, but it was never the intent of this series and as such these adjustments can be done at any point in time independently.

Reviewed-by: Erik Skultety <eskultet(a)redhat.com&gt;