enhancementhelp wanted
Repository metrics
- Stars
- (216 個のスター)
- PR merge metrics
- (PR metrics pending)
説明
Desired behavior
Per the Pimpl idiom, one would expect changes to the sdf::ParamPrivate class to not break ABI, but
since the class is in a public header (Param.h) and since it's members are used in function templates of sdf::Param, any changes made to sdf::ParamPrivate could break ABI.
This is similar to https://github.com/ignitionrobotics/sdformat/issues/607
Alternatives considered
n/a
Implementation suggestion
Provide accessor functions in sdf::Param so that the function templates do not need direct access to dataPtr. Then we can move sdf::ParamPrivate to Param.cc. This would be a breaking change, so it will have to target the main branch.