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

Refactor moss-deps for (increased) SafeD compliance #7

Open
3 tasks
ermo opened this issue Aug 15, 2022 · 1 comment
Open
3 tasks

Refactor moss-deps for (increased) SafeD compliance #7

ermo opened this issue Aug 15, 2022 · 1 comment

Comments

@ermo
Copy link
Member

ermo commented Aug 15, 2022

Currently, the when attempting to use SafeD, the compiler complains due to the way we use pointers to structs.

For the in-progress moss rewrite (based on the LMDB-based moss-db module), let's update moss-deps to have the following properties:

  • Fix @safe on all non-graph bits
  • Add global @trusted annotation on digraph code as a short-term workaround
  • Add suitable code-style notes from the experience, including thoughts on moving to a class-based approach for SafeD reasons.
@ermo ermo assigned ikeycode and unassigned ikeycode Aug 15, 2022
@ermo
Copy link
Member Author

ermo commented Aug 15, 2022

Due to unforeseen difficulties related to Dlang's Phobos library not being all the way SafeD yet, we've haxx0red our way around this w/@trusted block-level annotations to avoid blocking on it for the moss rewrite.

Leaving open but unassigned until such a time as Phobos' SafeD coverage is more mature.

@ikeycode ikeycode transferred this issue from another repository Mar 21, 2023
@ermo ermo added the moss-deps label Mar 21, 2023
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

3 participants