On 16 February 2012 17:39, Alexander Barabash
<alexander_barab...@mentor.com> wrote:
>
>    Add object_property_get_child().
>
>    Adding a direct accessor to a child property.
>
>    In the existing implementation, object_property_get() must be used,
>    with with a visitor, implementing the 'type_str' callback,
>    receiving the child's canonical path.
>
>    In the new implementation, the child is returned directly.
>    Link properties are resolved in the same way, as they are
>    when an absolute path is resolved.
>
>    Errors relevant to the operation, QERR_OBJECT_PROPERTY_NOT_FOUND
>    and QERR_OBJECT_PROPERTY_INVALID_TYPE were added.

Not trivial (look at the length of it!).

>    Also, in the new implementation, some common sense refactoring was done
>    in the file 'qom/object.c' in the code extracting child and link
> properties.

If you find yourself writing a commit message with a paragraph
starting "Also" this almost always means you should split it
into more than one patch.

-- PMM

Reply via email to