Markus Armbruster <arm...@redhat.com> writes: > Peter Maydell <peter.mayd...@linaro.org> writes: > >> On 7 June 2017 at 14:09, Markus Armbruster <arm...@redhat.com> wrote: >>> Peter Maydell <peter.mayd...@linaro.org> writes: >>>> So how should I obtain those semantics with this cleanup in place ? >>> >>> Two ways come to mind: >>> >>> * Define a PropertyInfo like qdev_prop_uint32 with a null >>> set_default_value(), and use that. >>> >>> * Add a flag to Property that makes qdev_property_add_static() skip >>> prop->info->set_default_value(), set it for your property. >>> >>> Actually, I'd probably do it the other way: call ->set_default_value() >>> only when the flag is set. No need to check it's non-null then. >>> Setting the flag when it's null is a programming error. >>> >>> Could one of these two work for you? >> >> The former would work but duplicates a pile of code from the >> qdev_prop_uint32 implementation. The latter would be cleaner >> but introduces a dependency on this series -- how close is it >> to hitting master? > > v2 was close, v3 just hit the list, but I'll be on vacation next week. > Wrapping it up this week seems unrealistic. Soon after I'm back is my > best estimate.
Review of v3 went swimmingly, pull request is on the list.