[v1] Various cleanups

[PATCH 0/5] Various cleanups

Michal Privoznik posted 5 patches 2 years, 5 months ago

Diff against v1 v1 v1
Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1635872947.git.mprivozn@redhat.com

There is a newer version of this series

scripts/cocci-macro-file.h     |  2 +-
src/conf/node_device_conf.c    | 46 +++++++++++++---------------------
src/libxl/libxl_capabilities.c |  2 +-
src/util/virpcivpd.c           | 19 +++-----------
tests/virpcimock.c             |  2 +-
tests/virpcivpdtest.c          | 29 ++++++++++-----------
6 files changed, 39 insertions(+), 61 deletions(-)

Expand all Fold all

[PATCH 0/5] Various cleanups

Posted by Michal Privoznik 2 years, 5 months ago

I've been playing with cocci lately and noticed it had troubles parsing
some files. When I looked into them I had trouble parsing them as well.

Michal Prívozník (5):
  node_device_conf: Bring variables into loops
  virpcivpd: Bring variables into loops
  virpcivpdtest: Declare variables at multiple lines
  lib: Use G_N_ELEMENTS instead of sizeof()/sizeof()
  scripts: Properly declare g_auto() stub for cocci

 scripts/cocci-macro-file.h     |  2 +-
 src/conf/node_device_conf.c    | 46 +++++++++++++---------------------
 src/libxl/libxl_capabilities.c |  2 +-
 src/util/virpcivpd.c           | 19 +++-----------
 tests/virpcimock.c             |  2 +-
 tests/virpcivpdtest.c          | 29 ++++++++++-----------
 6 files changed, 39 insertions(+), 61 deletions(-)

-- 
2.32.0

Re: [PATCH 0/5] Various cleanups

Posted by Tim Wiederhake 2 years, 5 months ago

On Tue, 2021-11-02 at 18:10 +0100, Michal Privoznik wrote:
> I've been playing with cocci lately and noticed it had troubles
> parsing
> some files. When I looked into them I had trouble parsing them as
> well.
> 
> Michal Prívozník (5):
>   node_device_conf: Bring variables into loops
>   virpcivpd: Bring variables into loops
>   virpcivpdtest: Declare variables at multiple lines
>   lib: Use G_N_ELEMENTS instead of sizeof()/sizeof()
>   scripts: Properly declare g_auto() stub for cocci
> 
>  scripts/cocci-macro-file.h     |  2 +-
>  src/conf/node_device_conf.c    | 46 +++++++++++++-------------------
> --
>  src/libxl/libxl_capabilities.c |  2 +-
>  src/util/virpcivpd.c           | 19 +++-----------
>  tests/virpcimock.c             |  2 +-
>  tests/virpcivpdtest.c          | 29 ++++++++++-----------
>  6 files changed, 39 insertions(+), 61 deletions(-)
> 

Reviewed-by: Tim Wiederhake <twiederh@redhat.com>