Skip to content

Use NETWORK_GetFirstFreeID for the remaining sector thinkers, and only generate these IDs in server mode.

Sean Baggaley requested to merge topic/default/getfirstfreeid into branch/default

72fcd67e7061 (from 2008!) introduced NETWORK_GetFirstFreeID, but only used it for ceilings. This extends it to the remaining sector thinkers (floors, elevators, platforms, waggles, pillars, doors).

This comes about after playing MAP25 of Necromantic Thirst, which has some linedefs that lower about 500 floors at once; under Zandronum, triggerring these linedefs freezes the gamesim for several seconds (with somewhat comedic results online). With this patch, that is reduced to an almost imperceptible stutter.

I also added a changeset that makes it so the IDs are only populated in server mode -- previously, it checked NETWORK_InClientMode() == false, which would cause them to generate in singleplayer as well; since they are only used for networking, they are redundant there.

Edited by Sean Baggaley

Merge request reports

Loading