phaneron/Kelsidavis-WoWee
1
0
Fork 0
mirror of https://github.com/Kelsidavis/WoWee.git synced 2026-04-07 13:43:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fix: far same-map teleport blocked packet handler for 41 seconds

Browse source 
loadOnlineWorldTerrain() was called directly from the worldEntryCallback
inside the packet handler, running the 20s warmup loop synchronously.
This blocked ALL packet processing and froze the game for 20-41 seconds.

Now defers the world reload to pendingWorldEntry_ which is processed on
the next frame, outside the packet handler. Position and camera snap
immediately so the player doesn't drift at the old location.

The /y respawn report was actually a server-initiated teleport (possibly
anti-spam or area trigger) that hit this 41-second blocking path.
This commit is contained in:
Kelsi 2026-03-28 15:38:44 -07:00
parent 9666b871f8
commit 2f96bda6fa
1 changed files with 12 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/core/application.cpp
View file

@ -2446,11 +2446,19 @@ void Application::setupUICallbacks() {
bool farTeleport = (teleportDistSq > 500.0f * 500.0f);
if (farTeleport) {
// Far same-map teleport (hearthstone, etc.): do a full world reload
// with loading screen to prevent falling through unloaded terrain.
// Far same-map teleport (hearthstone, etc.): defer full world reload
// to next frame to avoid blocking the packet handler for 20+ seconds.
LOG_WARNING("Far same-map teleport (dist=", std::sqrt(teleportDistSq),
"), triggering full world reload with loading screen");
loadOnlineWorldTerrain(mapId, x, y, z);
"), deferring world reload to next frame");
// Update position immediately so the player doesn't keep moving at old location
renderer->getCharacterPosition() = renderPos;
if (renderer->getCameraController()) {
auto* ft = renderer->getCameraController()->getFollowTargetMutable();
if (ft) *ft = renderPos;
renderer->getCameraController()->clearMovementInputs();
renderer->getCameraController()->suppressMovementFor(1.0f);
}
pendingWorldEntry_ = PendingWorldEntry{mapId, x, y, z};
return;
}
LOG_INFO("Same-map teleport (map ", mapId, "), skipping full world reload");