[Scummvm-git-logs] scummvm master -> 8573485ffbe8315f53b7e9a58566fd905c283403

alexbevi noreply at scummvm.org
Sun Oct 18 01:14:15 UTC 2026


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:
8573485ffb HARVESTER: Preserve exit region depth targets outside room bounds


Commit: 8573485ffbe8315f53b7e9a58566fd905c283403
    https://github.com/scummvm/scummvm/commit/8573485ffbe8315f53b7e9a58566fd905c283403
Author: Alex Bevilacqua (alex at alexbevi.com)
Date: 2026-10-17T21:14:05-04:00

Commit Message:
HARVESTER: Preserve exit region depth targets outside room bounds

Fixes #17396

Assisted-by: Codex:gpt-6.1-sol

Changed paths:
    engines/harvester/player.cpp


diff --git a/engines/harvester/player.cpp b/engines/harvester/player.cpp
index c219947ac9a..5829792cb05 100644
--- a/engines/harvester/player.cpp
+++ b/engines/harvester/player.cpp
@@ -1025,7 +1025,7 @@ static void setMoveTargetInternal(const RoomSetupState &state, RoomPlayerState &
 	playerState.nextMovementTick = 0;
 	playerState.targetX = CLIP<int>(targetX, 0, 639);
 	playerState.targetBottomY = clampRoomMovementY(state, targetBottomY);
-	playerState.targetZ = clampRoomDepth(state, targetZ);
+	playerState.targetZ = targetZ;
 	resetPlayerMoveTargetProgress(playerState);
 	debugC(1, kDebugPlayer,
 		"Harvester: player move target room='%s' current=(%d,%d,z=%.2f) target=(%d,%d,z=%.2f)",
@@ -1448,7 +1448,7 @@ bool Player::isIdleAnimationExcludedRoom(const Common::String &roomName) {
 
 void Player::setMoveTarget(const RoomSetupState &state, int targetX, float targetZ) {
 	RoomPlayerState &playerState = _state;
-	setMoveTargetInternal(state, playerState, targetX, targetZ,
+	setMoveTargetInternal(state, playerState, targetX, clampRoomDepth(state, targetZ),
 		mapRoomDepthToScreenY(state, targetZ));
 }
 




More information about the Scummvm-git-logs mailing list