On Wed, Sep 21, 2016 at 04:50:52PM +0200, Michal Privoznik wrote:
On 21.09.2016 15:54, Martin Kletzander wrote:On Wed, Sep 14, 2016 at 08:39:05AM +0200, Peter Krempa wrote:The automatic nodeset would creep into the persistent definition for live VMs with automatic pinning which is incorrect. https://bugzilla.redhat.com/show_bug.cgi?id=1365779 Peter Krempa (4): conf: Introduce virDomainObjGetOneDefState qemu: domain: Add macro to simplify access to vm private dataI must say I don't like this one ^^, it's way more readable to just use the variable which can be just optimized out, but since we use that already in multiple places, meh ACK series, I guess.Agreed. It makes debugging way worse and also does not work well with autocomplete in my vim. Revert it please.
I would suggest removing all at the same time. The conversation about how some people don't like it, even though it doesn't matter, can happen in that series then.
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list