Thomas Huth <thuth@xxxxxxxxxx> writes: > These parameters are in the way for further refactoring (since they > use an underscore in the name which is forbidden in QAPI), so let's > remove these now that their deprecation period is over. Forbidden, but there's an exception mechanism, so this reason isn't compelling. I believe the actual reason is they are "too ugly and inflexible to drag them along into the QAPI world" (your words). Suggest: Dropping these deprecated parameters now simplifies further refactoring. > Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx> > Signed-off-by: Thomas Huth <thuth@xxxxxxxxxx> Patch looks good to me, so Reviewed-by: Markus Armbruster <armbru@xxxxxxxxxx>
