consai2r2_teleop/include/visibility_control.hは公式のものに置き換えるべき

[HansRobo]

https://github.com/ros/class_loader/blob/ros2/include/class_loader/visibility_control.hpp

公式のものが提供されているのでconsai2r2で保持するのではなくそちらを使うべき．

/opt/ros/dishing/include/class_loader/visibility_control.hppの存在は確認済み

[zohar0729]

今から作業します

[zohar0729]

作業完了しました

[ShotaAk]

visibility_control.hはパッケージごとに用意するようです。
そのため、visibility_control.hはconsai2_teleop内のものを使いたいです。

ros2/ros2#37

(１つにまとめる仕組みが欲しい！っていう意見もあるみたいですが、
大元のgccもprefixを変えて使えって書いてますからね。。。)
ament/ament_cmake#164

[ShotaAk]

@HansRobo @zohar0729 上記理由より、本件と#39 をクローズしても良いですか？

[HansRobo]

問題ないです

[MiraiHattori]

https://discourse.ros.org/t/add-visibility-control-h-to-the-api-as-standard-usage/11681/4

[MiraiHattori]

メモです．
windowsでは各プロジェクトごとにないと本当に困るようです．

The trouble comes when you want library A to depend on library B. In that case, you need to mark the symbols of library B as dllimport (so library A can call them), but you need to mark the public symbols of library A as dllexport. So you can’t just have one set of macros to accomplish this. This is why there are per-project macros.

[zohar0729]

はい、閉じてもらって大丈夫です、すみません