On 11/25/19 5:54 AM, Pavel Mores wrote:
> This new version mostly integrates Cole's comments about the second version.
> Refactoring and behaviour change are now separate commits.  Tests succeed for
> every individual patch in the series.
> 
> Pavel Mores (4):
>   qemu: default video device type selection algoritm moved into its own
>     function
>   qemu: prepare existing test for change of the default video device
>     type
>   qemu: the actual change of default video devide type selection
>     algorithm
>   qemu: added tests of the new default video type selection algorithm

Reviewed-by: Cole Robinson <crobi...@redhat.com>

and pushed now.

patch #3 still had some style issues I pointed out in the previous
review: using 'else' when it isn't necessary, because every block
returns. Better to use the style of qemuDomainDefaultNetModel which
saves having to do any nested logic. I'll be happy to review a follow up
cleanup patch for that

Minor bit: I would have squashed patch #2 + #3 together, maybe only
separating them if there was lots of test suite churn needed, but I'm
not sure if others agree.

Thanks,
Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to