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.

Sign up for GitHub

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

Jump to bottom

Remove unnecessary CFRelease call #98

Merged
merged 1 commit into from Jul 13, 2012
Merged

Remove unnecessary CFRelease call #98

merged 1 commit into from Jul 13, 2012

Conversation

dmarkow
Copy link
Contributor

@dmarkow dmarkow commented Jul 13, 2012

Per a support ticket discussion with Laurent while troubleshooting some iOS 6 issues:

It is not necessary to call CFRelease in RubyMotion, as even CFType objects are automatically managed by the runtime. Calling it will result in the object being double-freed, which can potentially cause a crash later on.

mattetti added a commit that referenced this pull request Jul 13, 2012
@mattetti
Merge pull request #98 from dmarkow/cfrelease_fix
2a0ff2e 
Remove unnecessary CFRelease call
@mattetti mattetti merged commit 2a0ff2e into rubymotion-community:master Jul 13, 2012
@mattetti
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dmarkow @mattetti