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

Add transition fork, decision is subclass #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stonesong
Copy link
Contributor

Add a transition fork, a transition with two branches outgoing, and a decision is a subclass of a fork. (yes is the first outgoing branch and no the second when drawing)

@stonesong stonesong changed the base branch from rename-to-state-machine-remove-coloring to master August 5, 2020 11:20
@@ -162,6 +141,27 @@ CDBlockObjectVisitor >> visitEnd: anEndState [
"
]

{ #category : #visiting }
CDBlockObjectVisitor >> visitFork: aFork [
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I said in the chat, a fork is a separation of control flow and decision a simple branch

yes incoming: self.
no := aBuilder resolve: flowName no: (aCollection last: 2).
no incoming: self
^ secondBranch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No was easy to understand,now it is not

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