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

Patches and bugfixes from upstream pull requests #2

Closed
wants to merge 13 commits into from

Conversation

meyerj
Copy link

@meyerj meyerj commented Feb 2, 2017

Most of them are from orocos-toolchain#135 or even already in master, but have never been merged into rdt-next.

meyerj and others added 9 commits November 21, 2016 10:38
…tivate()

The removed line triggered a failed assert in StateMachine::enableEvents().

Signed-off-by: Johannes Meyer <[email protected]>
Yielding in the run program of state INIT already triggers the global transition
to the FINI state, before the send event can be processed and trigger a transition
to NEXT. This patch adds a condition to the global transition that only evaluates to
true after the run program has finished.

Signed-off-by: Johannes Meyer <[email protected]>
… protection to StateMachine::activate()

Signed-off-by: Johannes Meyer <[email protected]>
… if the state machine is not active

smStatus member variable is not protected by the mutex in other methods, e.g. StateMachine::activate().

Signed-off-by: Johannes Meyer <[email protected]>
… branch toolchain-2.9-statemachine-execution-logic

Follow-up of 6537ae7 after the state machine execution logic patches have been implemented in 2015.
I am not sure why these checks have been missing in rdt.

This commit syncs the rdt version of RTT::scripting::StateMachine with upstream toolchain-2.9-statemachine-execution-logic.

Signed-off-by: Johannes Meyer <[email protected]>
…tantiation

The loop that removed SendHandleAlias variables from the StateMachineService could have skipped the
second one for the case the first two variables are SendHandles.

Signed-off-by: Johannes Meyer <[email protected]>
…nstructor to ready() member function

... so that it can be reused in corba_ipc_server without having initialized the Boost Test framework.

This is a regression from 167076a.

Signed-off-by: Johannes Meyer <[email protected]>
@snrkiwi
Copy link
Owner

snrkiwi commented Mar 19, 2017

Already merged to rdt-next

@snrkiwi snrkiwi closed this Mar 19, 2017
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