qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [PATCH 14/22] migration: Remove get_status() accessor


From: Juan Quintela
Subject: [Qemu-devel] Re: [PATCH 14/22] migration: Remove get_status() accessor
Date: Wed, 23 Feb 2011 10:18:43 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux)

Yoshiaki Tamura <address@hidden> wrote:
> 2011/2/23 Juan Quintela <address@hidden>:
>> It is only used inside migration.c, and fields on that struct are
>> accessed all around the place on that file.

>
> I agree to access s->state directly inside of migration.c, but I
> disagree to remove get_status() accessor right away.  We don't
> have strong motivations for doing that AFAIK.

Only user outside of migration.c was ui/spice-core.c, and it just wanted
to know if migration has finished at all.

At this point I was trying to isolate what other parts of MigrationState
are used externally.  That way, it gets easier to change that later.

At this point, only things used outside of migration.c are:
- write, clase, get_error: trivial to fix, just add setters for them.
- fd: that is not enterely trivial to fix.

Later, Juan.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]