Re: [PATCH v2 01/10] migration: Use OBJECT_DECLARE_SIMPLE_TYPE

Peter Xu <[email protected]> writes:

> Migration object's class has nothing special, switch to the newly
> introduced macro.
>
> Suggested-by: Daniel P. Berrang=C3=A9 <[email protected]>
> Signed-off-by: Peter Xu <[email protected]>
> ---
>  migration/migration.h | 9 +--------
>  migration/migration.c | 7 +++----
>  2 files changed, 4 insertions(+), 12 deletions(-)
>
> diff --git a/migration/migration.h b/migration/migration.h
> index 841f49b215..293ad60e07 100644
> --- a/migration/migration.h
> +++ b/migration/migration.h
> @@ -263,14 +263,7 @@ void fill_destination_postcopy_migration_info(Migrat=
ionInfo *info);
>=20=20
>  #define TYPE_MIGRATION "migration"
>=20=20
> -typedef struct MigrationClass MigrationClass;
> -DECLARE_OBJ_CHECKERS(MigrationState, MigrationClass,
> -                     MIGRATION_OBJ, TYPE_MIGRATION)
> -
> -struct MigrationClass {
> -    /*< private >*/
> -    DeviceClass parent_class;
> -};
> +OBJECT_DECLARE_SIMPLE_TYPE(MigrationState, MIGRATION);
>=20=20
>  struct MigrationState {
>      /*< private >*/
> diff --git a/migration/migration.c b/migration/migration.c
> index 074d3f2c69..278cad502a 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -297,7 +297,7 @@ void migration_object_init(void)
>  {
>      /* This can only be called once. */
>      assert(!current_migration);
> -    current_migration =3D MIGRATION_OBJ(object_new(TYPE_MIGRATION));
> +    current_migration =3D MIGRATION(object_new(TYPE_MIGRATION));
>=20=20
>      /*
>       * Init the migrate incoming object as well no matter whether
> @@ -3975,7 +3975,7 @@ static void migration_class_init(ObjectClass *klass=
, const void *data)
>=20=20
>  static void migration_instance_finalize(Object *obj)
>  {
> -    MigrationState *ms =3D MIGRATION_OBJ(obj);
> +    MigrationState *ms =3D MIGRATION(obj);
>=20=20
>      qapi_free_BitmapMigrationNodeAliasList(ms->parameters.block_bitmap_m=
apping);
>      qapi_free_strList(ms->parameters.cpr_exec_command);
> @@ -3993,7 +3993,7 @@ static void migration_instance_finalize(Object *obj)
>=20=20
>  static void migration_instance_init(Object *obj)
>  {
> -    MigrationState *ms =3D MIGRATION_OBJ(obj);
> +    MigrationState *ms =3D MIGRATION(obj);
>=20=20
>      ms->state =3D MIGRATION_STATUS_NONE;
>      ms->mbps =3D -1;
> @@ -4040,7 +4040,6 @@ static const TypeInfo migration_type =3D {
>       */
>      .parent =3D TYPE_DEVICE,
>      .class_init =3D migration_class_init,
> -    .class_size =3D sizeof(MigrationClass),
>      .instance_size =3D sizeof(MigrationState),
>      .instance_init =3D migration_instance_init,
>      .instance_finalize =3D migration_instance_finalize,

Reviewed-by: Fabiano Rosas <[email protected]>