qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 0/8] qom: Use qlit to represent property defaults


From: Paolo Bonzini
Subject: Re: [PATCH 0/8] qom: Use qlit to represent property defaults
Date: Tue, 10 Nov 2020 17:39:08 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0

On 09/11/20 22:25, Eduardo Habkost wrote:
Based-on: 20201104160021.2342108-1-ehabkost@redhat.com
Git branch: https://gitlab.com/ehabkost/qemu/-/commits/work/qdev-qlit-defaults

This extend qlit.h to support all QNum types (signed int,
unsigned int, and double), and use QLitObject to represent field
property defaults.

It allows us to get rid of most type-specific .set_default_value
functions for QOM property types.

Eduardo Habkost (8):
   qobject: Include API docs in docs/devel/qobject.html
   qnum: Make qnum_get_double() get const pointer
   qnum: QNumValue type for QNum value literals
   qnum: qnum_value_is_equal() function
   qlit: Support all types of QNums
   qlit: qlit_type() function
   qom: Make object_property_set_default() public
   qom: Use qlit to represent property defaults

  docs/devel/index.rst                  |   1 +
  docs/devel/qobject.rst                |  11 +++
  include/hw/qdev-properties-system.h   |   2 +-
  include/qapi/qmp/qlit.h               |  16 +++-
  include/qapi/qmp/qnum.h               |  47 ++++++++++-
  include/qapi/qmp/qobject.h            |  48 +++++++----
  include/qom/field-property-internal.h |   4 -
  include/qom/field-property.h          |  26 +++---
  include/qom/object.h                  |  11 +++
  include/qom/property-types.h          |  21 ++---
  hw/core/qdev-properties-system.c      |   8 --
  qobject/qlit.c                        |   4 +-
  qobject/qnum.c                        | 116 +++++++++++++++-----------
  qom/field-property.c                  |  27 ++++--
  qom/object.c                          |   2 +-
  qom/property-types.c                  |  36 ++------
  tests/check-qjson.c                   |  72 ++++++++++++++--
  17 files changed, 295 insertions(+), 157 deletions(-)
  create mode 100644 docs/devel/qobject.rst


Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Thanks,

Paolo




reply via email to

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