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 data
>
> I 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.