Maescool / Catacomb-Snatch

Mojang's Humble bundle source
Other
295 stars 99 forks source link

Multiplayer bugs #930

Closed Maescool closed 11 years ago

Maescool commented 12 years ago
Maescool commented 12 years ago

picking up turrets and stuff is also causing desync.. guess caused by new feature of @delvar

Delvar commented 12 years ago

ill look into it, i assume im missing sending a network event on pickup. ill checkout before my patch and see what i broke :)

Delvar commented 12 years ago

ok i can see the issue, when buying/picking up it works on client but is not sent to server or vica versa the thing bought stays on the floor.

Delvar commented 12 years ago

i reverted my code and the pickup worked fine so i am to blame. ill go fix it. i assume its due to the move of the pickup/carrying code.

Delvar commented 12 years ago

seems to be an issue with Highlighted code, the other server/client doesn't see that the enemy player is highlighted the shop at all or highlighted other entities. so when the entity is created and added to the world it isn't picked up as it doesn't get to that bit of code.

Flet commented 12 years ago

@Maescool uh, its using TCP only for network IO :)

Flet commented 12 years ago

also you're right about the console. It is not synced up at all and will break mp

Delvar commented 12 years ago

ok fixed the issue, 2 things i had to do 1. remove re-decliration of 'int team' from Turret 2. allow 'highlighted' from Other players not just local and move the local check to the draw marker.

Flet commented 12 years ago

this all fixed up now?

kylecbrodie commented 12 years ago

I think so. Almost all actions are synced now.

godleydemon commented 11 years ago

closing up due to the main issues being fixed with this