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