[PULL,01/18] qdev-properties.h: Explicitly set the default value for arraylen properties

Message ID 1500295494-8991-2-git-send-email-peter.maydell@linaro.org
State Accepted
Commit d9a7b125d6b8048a503d10117ee8b7cbc44cc889
Headers show
Series
  • target-arm queue
Related show

Commit Message

Peter Maydell July 17, 2017, 12:44 p.m.
In DEFINE_PROP_ARRAY, because we use a PropertyInfo (qdev_prop_arraylen)
which has a .set_default_value member we will set the field to a default
value. That default value will be zero, by the C rule that struct
initialization sets unmentioned members to zero if at least one member
is initialized. However it's clearer to state it explicitly.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Message-id: 1499788408-10096-2-git-send-email-peter.maydell@linaro.org
---
 include/hw/qdev-properties.h | 1 +
 1 file changed, 1 insertion(+)

-- 
2.7.4

Patch hide | download patch | download mbox

diff --git a/include/hw/qdev-properties.h b/include/hw/qdev-properties.h
index f6692d5..12c0c7b 100644
--- a/include/hw/qdev-properties.h
+++ b/include/hw/qdev-properties.h
@@ -111,6 +111,7 @@  extern const PropertyInfo qdev_prop_link;
                           _arrayfield, _arrayprop, _arraytype) {        \
         .name = (PROP_ARRAY_LEN_PREFIX _name),                          \
         .info = &(qdev_prop_arraylen),                                  \
+        .defval.u = 0,                                                  \
         .offset = offsetof(_state, _field)                              \
             + type_check(uint32_t, typeof_field(_state, _field)),       \
         .arrayinfo = &(_arrayprop),                                     \