On 02/17/2016 12:17 PM, Cole Robinson wrote:
Seems like the natural fit, since we are already adding other XML
bits
in the PostParse routine.
Previously AddImplicitControllers was only called at the end of XML
parsing, meaning code that builds a DomainDef by hand had to manually
call it. Now those PostParse callers get it for free.
There's some test churn here; xen xm and sexpr test suite bits weren't
calling this before, but now they are, so you'll see new IDE controllers.
I don't think this will cause problems in practice, since the code already
needs to handle these implicit controllers like in the case when a user
defines their own XML.
---
cc: jfehlig(a)suse.com
Jim, does that reasoning for the xen churn sound good to you?
A general ACK from me, since what you're doing makes sense and should be
a NOP, but I'll leave it to Jim to ACK the change in xen test cases.