qemu_monitor_json: Use g_autoptr() for virJSONValue

Browse Source

A lot of explicit free calls can be saved when virJSONValue
variables are declared with g_autoptr(). There's one caveat:
there was a slight deviation from our usual pattern such that
@cmd variable was not initialized to NULL but as the very first
step it was assigned a value using qemuMonitorJSONMakeCommand().
While this works in theory it upset my GCC-11.2 (but only when
building with -O2). So I had to initialize the variable in such
case too.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Tim Wiederhake <twiederh@redhat.com>

...

This commit is contained in:

Michal Privoznik 2021-10-21 10:14:47 +02:00

parent 40bbcb47cc

commit 551f91b4cb

1 changed files with 227 additions and 396 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Show Stats Download Patch File Download Diff File Expand all files Collapse all files

623

src/qemu/qemu_monitor_json.c

View File

File diff suppressed because it is too large Load Diff