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

[RubyMotion support] Fixing some issues with Proc based guardians and callbacks. #318

Merged
merged 2 commits into from
Feb 9, 2016
Merged

[RubyMotion support] Fixing some issues with Proc based guardians and callbacks. #318

merged 2 commits into from
Feb 9, 2016

Conversation

Infotaku
Copy link
Contributor

@Infotaku Infotaku commented Feb 5, 2016

Spotted an issue due to RubyMotion's Proc implementation.

Proc does not have the method 'parameters' nor 'source_location'.
This was going to be a pain in my ass until I discovered that Proc's 'arity' method had a different behavior in Ruby and RubyMotion.. at my advantage !
I had to sacrify some debugging 'hints' for us RubyMotion developers to achieve the fix, but nothing too hard.

…hod.

This may have some impact on debugging for RubyMotion, I would recommend using symbols to define guardians and callbacks whenever possible.
@Infotaku
Copy link
Contributor Author

Infotaku commented Feb 5, 2016

I just saw that you have a PR (#316) that adds some logic to the same method.
By what I see in the edited files, this PR will re-break RubyMotion support (use of 'source_location').

Please tell me if you need me to make a pass on that logic to bring RubyMotion support back after the merge.

@Infotaku Infotaku mentioned this pull request Feb 5, 2016
@alto alto self-assigned this Feb 9, 2016
@alto
Copy link
Member

alto commented Feb 9, 2016

@Infotaku Thanks for your work on this. I will let you know once #316 is merged into master (and before I release it).

alto added a commit that referenced this pull request Feb 9, 2016
[RubyMotion support] Fixing some issues with Proc based guardians and callbacks.
@alto alto merged commit 54f4cfb into aasm:master Feb 9, 2016
alto added a commit that referenced this pull request Feb 9, 2016
@alto
Copy link
Member

alto commented Feb 9, 2016

@Infotaku I just merged #316 into master. Please have a look! I will release it once you say it's okay for you.

@Infotaku Infotaku deleted the RubyMotion branch February 9, 2016 13:43
@alto
Copy link
Member

alto commented Feb 22, 2016

I just released version 4.9.0 including the change.

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