qemu_alias: Refactor functions

[PATCH v2 0/2] qemu_alias: Refactor functions

Kristina Hanicova posted 2 patches 2 years, 4 months ago

Download series mbox

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(-)

Expand all Fold all

[PATCH v2 0/2] qemu_alias: Refactor functions

Posted by Kristina Hanicova 2 years, 4 months 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 2 years, 4 months 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