Alessandro

Results 13 comments of Alessandro

Absolutely agree, I had the same problems, Pipeline is not intended to work like ViaVersion actually does and packet transformations should be done on a separate channel.

> I tried to update this patch, no change at all in tps display. > > Adding debug sysout in loop shows that server is ticking 51 - 52 ms,...

![2024-05-01_13 06 06](https://github.com/ViaVersion/ViaRewind/assets/21027982/94b64801-8d98-4617-ab99-8b6d17ad8fbe) Now I don't crash but I'm able to see zombies inside Holograms. I'm using DecentHolograms plugin for holograms.

The issue is caused by #383

@aromaa any news about this? I think it is quite embarrassing to have to do all that for a simple data loading/unloading of the player in a safe way (which...

I looked more closely at the code, and I'd have something to say about this: > Fixing just the disconnection isn't enough as the connection registration is done after `LoginEvent`...

Oh, you're right. Then that has to be adjusted too. Maybe the idea of having a duplicate register connection check was to avoid unnecessary calls later. I think these two...

I don't see any issues in **applying the Disconnect fix patch you mentioned**, by moving the `DisconnectEvent` call before `unregisterConnection`, and then using this code: ``` private ConcurrentMap usersByUuid; public...