[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] vmstate: check subsection_found is enough
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v2] vmstate: check subsection_found is enough |
Date: |
Tue, 14 May 2019 14:40:44 +0100 |
User-agent: |
Mutt/1.11.4 (2019-03-13) |
* Wei Yang (address@hidden) wrote:
> subsection_found is true implies vmdesc is not NULL.
>
> This patch remove the additional check on vmdesc and rename
> subsection_found to vmdesc_has_subsections to make it more self-explain.
>
> Signed-off-by: Wei Yang <address@hidden>
Queued.
> ---
> v2:
> * rename it to vmdesc_has_subsections
> ---
> migration/vmstate.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/migration/vmstate.c b/migration/vmstate.c
> index e2bbb7b5f7..1305d1a528 100644
> --- a/migration/vmstate.c
> +++ b/migration/vmstate.c
> @@ -496,7 +496,7 @@ static int vmstate_subsection_save(QEMUFile *f, const
> VMStateDescription *vmsd,
> void *opaque, QJSON *vmdesc)
> {
> const VMStateDescription **sub = vmsd->subsections;
> - bool subsection_found = false;
> + bool vmdesc_has_subsections = false;
> int ret = 0;
>
> trace_vmstate_subsection_save_top(vmsd->name);
> @@ -508,9 +508,9 @@ static int vmstate_subsection_save(QEMUFile *f, const
> VMStateDescription *vmsd,
> trace_vmstate_subsection_save_loop(vmsd->name, vmsdsub->name);
> if (vmdesc) {
> /* Only create subsection array when we have any */
> - if (!subsection_found) {
> + if (!vmdesc_has_subsections) {
> json_start_array(vmdesc, "subsections");
> - subsection_found = true;
> + vmdesc_has_subsections = true;
> }
>
> json_start_object(vmdesc, NULL);
> @@ -533,7 +533,7 @@ static int vmstate_subsection_save(QEMUFile *f, const
> VMStateDescription *vmsd,
> sub++;
> }
>
> - if (vmdesc && subsection_found) {
> + if (vmdesc_has_subsections) {
> json_end_array(vmdesc);
> }
>
> --
> 2.19.1
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2] vmstate: check subsection_found is enough,
Dr. David Alan Gilbert <=