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

use gen_statem:start/3 since gen_fsm:start/3 is deprecated #11

Closed
wants to merge 2 commits into from
Closed

Conversation

m13m
Copy link

@m13m m13m commented Feb 9, 2018

gen_fsm:start/3 is deprecated and replaced by gen_statem:start/3

@manuel-rubio
Copy link
Member

@m13m you still need to change the behaviour entry (line 4) and maybe improve the return values to use the new ones. But I think this approach could be enough even to keep backwards compatibility creating a macro.

manuel-rubio pushed a commit that referenced this pull request Feb 9, 2018
@manuel-rubio
Copy link
Member

I've added a change to use gen_statem in OTP 19+ without break the compatibility with OTP 17 and OTP 18.

@manuel-rubio
Copy link
Member

I've created #12 to continue working on the upgrade. There are more changes should be added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants