[libvirt PATCH 0/5] Some fixes in domain_conf.c: virDomainFeaturesDefParse

Tim Wiederhake posted 5 patches 3 years, 1 month ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210304083531.9690-1-twiederh@redhat.com
src/conf/domain_conf.c | 36 ++++++++++++++----------------------
1 file changed, 14 insertions(+), 22 deletions(-)
[libvirt PATCH 0/5] Some fixes in domain_conf.c: virDomainFeaturesDefParse
Posted by Tim Wiederhake 3 years, 1 month ago
Patches 1-4 are minor cleanups, patch 5 is the actual one that I want out
of the way before continuing down my rabbit hole of the week.

Tim Wiederhake (5):
  domain_conf: Reduce scope of tmp in virDomainFeaturesDefParse
  domain_conf: Reduce scope of gic_version in virDomainFeaturesDefParse
  domain_conf: Reduce scope of node in virDomainFeaturesDefParse
  domain_conf: Add missing break in switch
  domain_conf: Fix check for hyperv stimer

 src/conf/domain_conf.c | 36 ++++++++++++++----------------------
 1 file changed, 14 insertions(+), 22 deletions(-)

-- 
2.26.2


Re: [libvirt PATCH 0/5] Some fixes in domain_conf.c: virDomainFeaturesDefParse
Posted by Michal Privoznik 3 years, 1 month ago
On 3/4/21 9:35 AM, Tim Wiederhake wrote:
> Patches 1-4 are minor cleanups, patch 5 is the actual one that I want out
> of the way before continuing down my rabbit hole of the week.
> 
> Tim Wiederhake (5):
>    domain_conf: Reduce scope of tmp in virDomainFeaturesDefParse
>    domain_conf: Reduce scope of gic_version in virDomainFeaturesDefParse
>    domain_conf: Reduce scope of node in virDomainFeaturesDefParse
>    domain_conf: Add missing break in switch
>    domain_conf: Fix check for hyperv stimer
> 
>   src/conf/domain_conf.c | 36 ++++++++++++++----------------------
>   1 file changed, 14 insertions(+), 22 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

and pushed.

Michal