-
-
Notifications
You must be signed in to change notification settings - Fork 735
Make KRR allocate @safe #6416
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
Make KRR allocate @safe #6416
Conversation
Thanks for your pull request and interest in making D better, @edi33416! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please see CONTRIBUTING.md for more information. If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment. Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub fetch digger
dub run digger -- build "master + phobos#6416" |
Great the GC is segfaulting here:
Opened an issue -> https://issues.dlang.org/show_bug.cgi?id=18720 and restarted. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please mind @jercaianu 's review
3b4e48c
to
743aeff
Compare
Added @safe
to functions and tests
(rebased) |
@thewilsonator you wanted this one. |
No description provided.