gazebosim/sdformat

Changes to sdf::ParamPrivate should not break ABI

開放

#773 建立於 2021年12月6日

 (0 則留言) (1 個反應) (0 位負責人)C++ (122 個分叉)auto 404
enhancementhelp wanted

倉庫指標

星標
 (216 顆星)
PR 合併指標
 (PR 指標待抓取)

描述

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.

貢獻者指南