清水様
安藤です
ありがとうございます。
ご指摘のManager::createContext()ですが、実はこの関数は
テンポラリなもので、release版では coil::GlobalFactory を
使用する予定です。
> OpenRTM-aist開発者の皆様
>
> 1.0.0で、
> Manager::createContext()
> が追加されましたが、
> これの逆に当たるオペレーションが
> 無いように思います。
>
> Manager::destroyContext(ExecutionContextBase*);
> のようなものが必要ではないでしょうか？
>
> これに関連して、
> RTObject_impl::finalizeContexts();
> の中でECが直接deleteされているのですが、
> factoryを介さないdestroyは、
> createとの整合性が崩れるので
> やめた方がよいと思います。
>
> # 1.0系は品質が重要と思いますので、
> 敢えて厳しくソースを眺めています。
> 気づいた点はどんどん報告していきますので、
> よろしくご検討ください。
>
> 静岡大 清水
OpenRTM-aist開発者の皆様
1.0.0で、
Manager::createContext()
が追加されましたが、
これの逆に当たるオペレーションが
無いように思います。
Manager::destroyContext(ExecutionContextBase*);
のようなものが必要ではないでしょうか？
これに関連して、
RTObject_impl::finalizeContexts();
の中でECが直接deleteされているのですが、
factoryを介さないdestroyは、
createとの整合性が崩れるので
やめた方がよいと思います。
# 1.0系は品質が重要と思いますので、
敢えて厳しくソースを眺めています。
気づいた点はどんどん報告していきますので、
よろしくご検討ください。
静岡大 清水