search

OpenRTM / OpenRTM-aist

OpenRTM-aist: RT-Middleware and OMG RTC implementation in C++ implemented by AIST
https://openrtm.org/
Other
19 stars 12 forks source link

PublisherPeriodic::invokeListener() 内で 実引数がPORT_OKの場合に戻り値がPORT_ERROR になる #545

Closed tonboAkinori closed 5 years ago

tonboAkinori commented 5 years ago

質問：実引数が PORT_OK の場合に PORT_ERROR を返す処理になっているが問題ないか

実コードの関数内のコメントを見ると、switch文の前に PORT_OK のコメント記載もあり PORT_OK を返す処理の漏れでないか確認したい。 なお、履歴を確認したが GitHub に移行した後に該当関数の処理変更は無かった。

https://github.com/OpenRTM/OpenRTM-aist/blob/788de6745af74282c3815677ba38fc14d002e403/src/lib/rtm/PublisherPeriodic.cpp#L562-L595

もう一箇所同じような処理があったので追記。

https://github.com/OpenRTM/OpenRTM-aist/blob/788de6745af74282c3815677ba38fc14d002e403/src/lib/rtm/PublisherNew.cpp#L543-L576

質問背景

警告オプションの switch-enum に対する対応中に気になり起票した。

switch-enum オプションは enum値を使った分岐処理で未使用の定義があると default ラベルがあっても未使用のenum値があると警告される。

tonboAkinori commented 5 years ago

回答頂いた内容を記載しておく

この関数はエラー通知として使われているため
PORT_OK が実引数となるという使われ方をしない。 

コード内のコメントを修正だけする。 PORT_OK 部分だけ。