Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

checks if component is already activated/deactivated #1024

Merged
merged 1 commit into from
Jul 24, 2016

Conversation

fkanehiro
Copy link
Owner

No description provided.

@k-okada
Copy link
Contributor

k-okada commented Jul 21, 2016

tork-a/rtmros_nextage#248 の問題を回避しようとしている,という理解でいいでしょうか.

@fkanehiro
Copy link
Owner Author

237cdc9 の変更で、すでにスタートしているRTCを再度スタートしようとしたり、すでにストップしているRTCを再度ストップしようとするとエラーとみなされる(Falseを返す)ようになったのをそれらは通す(Trueを返す)ようにしています。startやstopの返り値を見ていなければ、赤字のメッセージが出るだけで実害はなかったはずですが、見ていると影響があったかと思います。

@k-okada
Copy link
Contributor

k-okada commented Jul 21, 2016

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2676/
Test PASSed.

@fkanehiro fkanehiro merged commit 0e36572 into master Jul 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants