Jim Meyering <jim@xxxxxxxxxxxx> wrote: > "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote: >> This patch ensures all public API methods only have a single exit >> path, to make mutex unlocking simpler. >> >> remote_internal.c | 1256 +++++++++++++++++++++++++++++++++++------------------- > > Hi Dan, > > I got about halfway through it and spotted something odd: Other than that, the rest of that patch looks fine. But oh-so-boring to read (and write, I'm sure). At about 25% of the way through I even considered writing something to parse the 2 or 3 main patterns and automate the process. But there would have been too many little irregularities... -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list