[PATCH v2 0/2] qemu_alias: Refactor functions

Kristina Hanicova posted 2 patches 1 week, 6 days ago
Failed in applying to current master (apply log)
src/qemu/qemu_alias.c   | 185 +++++++++++++++-------------------------
src/qemu/qemu_alias.h   |  44 +++++-----
src/qemu/qemu_hotplug.c |  43 ++++------
3 files changed, 105 insertions(+), 167 deletions(-)

[PATCH v2 0/2] qemu_alias: Refactor functions

Posted by Kristina Hanicova 1 week, 6 days ago
This is v2 of: https://listman.redhat.com/archives/libvir-list/2021-November/msg00748.html

Diff to v1:
* changed return type of functions if possible - when function
  always returned success (suggested by Martin)

Kristina Hanicova (2):
  qemu_alias: Rewrite of code pattern
  qemu_alias: change return type to void if possible

 src/qemu/qemu_alias.c   | 185 +++++++++++++++-------------------------
 src/qemu/qemu_alias.h   |  44 +++++-----
 src/qemu/qemu_hotplug.c |  43 ++++------
 3 files changed, 105 insertions(+), 167 deletions(-)

-- 
2.31.1

Re: [PATCH v2 0/2] qemu_alias: Refactor functions

Posted by Michal Prívozník 1 week, 6 days ago
On 11/24/21 17:12, Kristina Hanicova wrote:
> This is v2 of: https://listman.redhat.com/archives/libvir-list/2021-November/msg00748.html
> 
> Diff to v1:
> * changed return type of functions if possible - when function
>   always returned success (suggested by Martin)
> 
> Kristina Hanicova (2):
>   qemu_alias: Rewrite of code pattern
>   qemu_alias: change return type to void if possible
> 
>  src/qemu/qemu_alias.c   | 185 +++++++++++++++-------------------------
>  src/qemu/qemu_alias.h   |  44 +++++-----
>  src/qemu/qemu_hotplug.c |  43 ++++------
>  3 files changed, 105 insertions(+), 167 deletions(-)
> 

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

Michal