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

Fix numerous problems with the Entity Manager for multiplayer #32

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix numerous problems with the Entity Manager for multiplayer #32

wants to merge 2 commits into from

Conversation

andrewvy
Copy link
Owner

This will be a running PR for fixing up the multiplayer aspects of HHVMCraft.

  • send entities that the player comes in range to
  • use the entity UUID in places where we use it as a key -> value

Issues:

  • Cannot properly update/set the spawned entity position. (stuck at 0,0,0)
  • EntityManager gets null entities sometimes.

@andrewvy
Fix numerous problems with the Entity Manager for multiplayer
80ac3d8 
- send entities that the player comes in range to
- use the entity UUID in places where we use it as a key -> value
@andrewvy andrewvy self-assigned this Feb 27, 2016
@andrewvy
Examine the connection to see if it's writable before writing packets
0ada3bc 
- ChunkRadius is used in the updateChunk method to load chunks in a radius around the player
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@andrewvy andrewvy

Labels
in-progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@andrewvy