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.

Sign up for GitHub

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

Jump to bottom

RemoveEntity at System level #56

Closed
EtienneBruines opened this issue Nov 5, 2015 · 1 comment
Closed

RemoveEntity at System level #56

EtienneBruines opened this issue Nov 5, 2015 · 1 comment
Labels
bug

Comments

@EtienneBruines
Copy link
Member

@matiwinnetou and I talked about this a few days ago; RemoveEntity is now implemented at World, which removes it from the global list of Entitys, but it also needs to be removed from the specific Systems that update the Entity.
@EtienneBruines EtienneBruines mentioned this issue Nov 5, 2015
Closed
@paked
Copy link
Member

paked commented Nov 5, 2015

Ah. That needs to be fixed,

EtienneBruines added a commit to EtienneBruines/engi that referenced this issue Nov 6, 2015
@EtienneBruines
Fixed bugs
34dc764 
Systems now have a map of entities - note that &System{} has now been
replaced by NewSystem() - to initialize the map.

engi now has a SetTitle function exposed, which sets the window title.

RenderSystem now also stops rendering stuff that isn't part of the
RenderSystem any more.

Fixes EngoEngine#56 and fixes EngoEngine#57.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@EtienneBruines @paked