-
Notifications
You must be signed in to change notification settings - Fork 373
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
iox-#341 implemented condition variable cleanup in waitset destructor #443
iox-#341 implemented condition variable cleanup in waitset destructor #443
Conversation
…tset destructor Signed-off-by: Christian Eltzschig <me@elchris.org>
@@ -49,7 +49,7 @@ class iox_user_trigger_test : public Test | |||
iox_user_trigger_storage_t m_sutStorage; | |||
iox_user_trigger_t m_sut; | |||
|
|||
ConditionVariableData m_condVar; | |||
ConditionVariableData m_condVar{"Horscht"}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😁👍
|
||
auto interfacePointer = m_portManager->acquireInterfacePortData(iox::capro::Interfaces::INTERNAL, "itfPenguin"); | ||
EXPECT_THAT(interfacePointer, Eq(nullptr)); | ||
EXPECT_TRUE(errorHandlerCalled); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in the PublisherOverflow
the correct error code is also tested. Shouldn't this be done here too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not possible since this returns just a pointer and not an expected. There is a todo in the code to change this.
{ | ||
std::vector<iox::popo::ConditionVariableData*> condVarContainer; | ||
|
||
std::string process = "HypnoToadForEver"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all glory to the hypnotoad
Signed-off-by: Christian Eltzschig me@elchris.org
Pre-Review Checklist for the PR Author
iox-#123-this-is-a-branch
)iox-#123 commit text
)git commit -s
)Notes for Reviewer
Checklist for the PR Reviewer
Post-review Checklist for the PR Author
Post-review Checklist for the Eclipse Committer
References