Is this pull request associated with an issue(s)?
Fixes #129 and #128
Description
This PR fixes two duplicated test section names discovered by https://github.com/CMakePP/CMakeTest/pull/94
In doing so, however, a bug was revealed where inherited attributes did not have their default values. This PR also fixes that issue.
Side note
There appears to be code in _cpp_object_get_attr_guts that assumes non-flattened attributes and that only the attributes defined in the class itself would be stored in the attrs meta-attribute. Is there a particular reason flattening the arguments was needed?
Is this pull request associated with an issue(s)? Fixes #129 and #128
Description This PR fixes two duplicated test section names discovered by https://github.com/CMakePP/CMakeTest/pull/94 In doing so, however, a bug was revealed where inherited attributes did not have their default values. This PR also fixes that issue.
Side note There appears to be code in
_cpp_object_get_attr_guts
that assumes non-flattened attributes and that only the attributes defined in the class itself would be stored in theattrs
meta-attribute. Is there a particular reason flattening the arguments was needed?