Unvanquished icon indicating copy to clipboard operation
Unvanquished copied to clipboard

Don't transmit some fake entities to clients

Open slipher opened this issue 1 year ago • 2 comments

Don't waste bandwidth on entities that don't do anything for clients.

Hearing about layouts which cause a buffer overflow when they are the built-in layout but not when loaded from a layout file (or vice versa, don't remember which way it was), it occurs to me that this might be related to the fake entities used to spawn buildables. They are known to spawn at different times when a layout file is used, resulting in different entity numbering. So you guys can give this a shot and see if it allows bigger layouts to load

slipher avatar Sep 25 '24 17:09 slipher

It does not allow the layout in question to load.

DolceTriade avatar Sep 25 '24 18:09 DolceTriade

I found the issue describing the discrepancy: https://github.com/Unvanquished/Unvanquished/issues/2124. I had it wrong in the OP, the difference caused by whether the layout is loaded following a map_restart or not, not whether it uses a layout file. This is because a new entity baseline is not created after map_restart.

Anyway it's still a good idea not to send useless stuff, regardless of whether it fixes a bug.

slipher avatar Sep 25 '24 20:09 slipher