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

Remove lots of force unwrapping #745

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

Conversation

rsmartatl
Copy link

@rsmartatl rsmartatl commented Dec 14, 2020

and instead guard for nil values at top of function or other protections that wont crash embedding programs, although new behaviours may not be optimal they are better than a crash.

@rsmartatl
Copy link
Author

@ystrot hey mate. I guess I might have missed part of the process to put this on your radar but we'd really like to ensure the framework doesn't accidentally crash the app with a force unwrap. Would you mind having a look at this?

@vox-humana
Copy link
Contributor

Just to add more context. This is follow up of #732 discussion and this previously merged PR #733

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