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

Simplify types for one system #15

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

Conversation

Hjaltesorgenfrei
Copy link
Owner

@Hjaltesorgenfrei Hjaltesorgenfrei commented Feb 2, 2024

Changes to use the ISystem directly instead of the Type Dependent System, which had the intention of keeping track what is written/read from in a system. Doing this would allow type system guarenteed multithreading.
This draft is merely an example of a single system converted to using manual booking keeping of read/writes.

Pros:

  • Faster Compile, the templates seems to take a significant time to compile.
  • Simplere to expand, i for example want to add stuff like creates/delete entities and writes to a component for next frame.

Cons:

  • Keeping track of what components is written and read is the responsible of the developer. Which may lead to race conditions when multithreading is introduced.

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.

1 participant