Re: [PATCH 00/16] Misc cleanups

[Paolo Bonzini <pbonzini@redhat.com>]

Looks good apart from the two patches I comment on.

Paolo

Il 4 maggio 2022 19:30:09 CEST, marcandre.lureau@redhat.com ha scritto:
>From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
>Hi,
>
>Perhaps the last series of preliminary patches before I propose a longer series
>to add qemu-common & qga meson subprojects. That's why they are mostly
>QGA-related cleanups.
>
>Thanks for the reviews!
>
>Marc-André Lureau (16):
>  include: move qemu_*_exec_dir() to cutils
>  util/win32: simplify qemu_get_local_state_dir()
>  tests: make libqmp buildable for win32
>  compiler.h: add QEMU_{BEGIN,END}_IGNORE_INITIALIZER_OVERRIDES
>  qobject/json-lexer: disable -Winitializer-overrides warnings
>  include: adjust header guards after renaming
>  qga: flatten safe_open_or_create()
>  osdep: export qemu_open_cloexec()
>  qga: use qemu_open_cloexec() for safe_open_or_create()
>  qapi/error: add g_autoptr(Error) support
>  qga: replace qemu_open_old() with qemu_open_cloexec()
>  test/qga: use G_TEST_DIR to locate os-release test file
>  qga/wixl: prefer variables over environment
>  qga/wixl: require Mingw_bin
>  qga/wixl: simplify some pre-processing
>  qga/wixl: replace QEMU_GA_MSI_MINGW_BIN_PATH with glib bindir
>
> configure                            |   9 +-
> meson.build                          |   5 +-
> include/qapi/error.h                 |   2 +
> include/qemu/compiler.h              |  11 ++
> include/qemu/cutils.h                |   7 ++
> include/qemu/help-texts.h            |   4 +-
> include/qemu/osdep.h                 |   9 +-
> tests/qtest/libqmp.h                 |   2 +
> qemu-io.c                            |   1 +
> qga/channel-posix.c                  |  18 ++--
> qga/commands-posix.c                 | 146 +++++++++++++--------------
> qobject/json-lexer.c                 |   4 +
> storage-daemon/qemu-storage-daemon.c |   1 +
> tests/qtest/fuzz/fuzz.c              |   1 +
> tests/qtest/libqmp.c                 |  35 +++++--
> tests/unit/test-qga.c                |  11 +-
> util/cutils.c                        | 108 ++++++++++++++++++++
> util/osdep.c                         |  10 +-
> util/oslib-posix.c                   |  81 ---------------
> util/oslib-win32.c                   |  53 +---------
> qga/installer/qemu-ga.wxs            |  83 +++++----------
> qga/meson.build                      |  11 +-
> 22 files changed, 313 insertions(+), 299 deletions(-)
>