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

Safety 0 is a bad idea #30

Open
slyrus opened this issue Mar 29, 2017 · 1 comment
Open

Safety 0 is a bad idea #30

slyrus opened this issue Mar 29, 2017 · 1 comment

Comments

@slyrus
Copy link
Member

slyrus commented Mar 29, 2017

Turning safety to 0 for so much of the code is a bad idea, and can mask real bugs. See next issue...

@varjagg
Copy link
Collaborator

varjagg commented Mar 29, 2017

It is, however other safety levels result in unacceptable performance penalties last time I tried. It's probably a combination of me being not smart enough to declare types properly and of SBCL unable to infer them.

It's a good idea to reduce them in scope to critical places, but is one of the things I haven't gotten around to yet.

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

No branches or pull requests

2 participants