[Scummvm-git-logs] scummvm master -> a1a37aa2062ba1da4bcae2cd10887240c1934051

[OMGPizzaGuy]

This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://api.github.com/repos/scummvm/scummvm .

Summary:
a1a37aa206 ULTIMA8: Fix actors placed inside objects when a large step hits them


Commit: a1a37aa2062ba1da4bcae2cd10887240c1934051
    https://github.com/scummvm/scummvm/commit/a1a37aa2062ba1da4bcae2cd10887240c1934051
Author: pmanyeh (32324116+pmanyeh at users.noreply.github.com)
Date: 2026-10-21T18:07:35-05:00

Commit Message:
ULTIMA8: Fix actors placed inside objects when a large step hits them

AnimationTracker::step passed the end point as the start of the sweep
interpolation, so an actor whose jump hit a wall close to its start was
moved almost to the end point, inside the wall. Every animation then
failed at once and the avatar mover was woken again in the same tick
until the kernel killed the processes (the game seemed to hang), or
the avatar fell below the map.

This bug is regression from commit 06df655, where the wrong parameter
was removed from GetInterpolatedCoords().

Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>

Changed paths:
    engines/ultima/ultima8/world/actors/animation_tracker.cpp


diff --git a/engines/ultima/ultima8/world/actors/animation_tracker.cpp b/engines/ultima/ultima8/world/actors/animation_tracker.cpp
index 555d8015c42..21f813e6874 100644
--- a/engines/ultima/ultima8/world/actors/animation_tracker.cpp
+++ b/engines/ultima/ultima8/world/actors/animation_tracker.cpp
@@ -286,7 +286,7 @@ bool AnimationTracker::step() {
 				}
 #endif
 				_blocked = true;
-				_curr = collision.GetInterpolatedCoords(end, start);
+				_curr = collision.GetInterpolatedCoords(start, end);
 				return false;
 			}
 		}