Opened 4 years ago

Last modified 10 days ago

#11841 pending defect (pending)

GRIM: Game locks if the user opens the inventory while Manny is knocking at Glottis' door

Reported by: sev- Owned by:
Priority: high Component: Engine: Grim
Version: Keywords:
Cc: Game: Grim Fandango

Description

If the user asks Manny to knock at Glottis' garage door, and while he is approaching the door the user quickly opens the inventory, the game locks.

It's due to a bug in the scripts, since it happens also with the original executable


This is a lot larger of an issue than just using that one door. They have a fairly common pattern in the scripts (used 73 times).

if (walk manny to the object) {

disable input;
dialogue;
enable input;

}
The walk Manny call will return true when Manny makes it to the object and false if he gets blocked. The possible fixes I see are to to disable the inventory key while walking to an object, or make the inventory key cancel the walking.


I'll just leave this here.


https://github.com/residualvm/residualvm/issues/694

Attachments (1)

197315fe-c1d6-11e3-97d0-d5d694e6430f.png (290.4 KB ) - added by sev- 4 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 by mparnaudeau, 3 years ago

Trying to reproduce this bug, I get an error ImSetParam: getting name from number is not supported! and the debugger is called. It happens when I open the inventory (keypress i) whatever if Manny is walking or not, as long as he is in the garage. Is it the same problem?

comment:2 by somaen, 2 weeks ago

Priority: normalhigh

Would be good to resolve this for the 2.9.0 release.

comment:3 by aquadran, 2 weeks ago

I would rather avoid patching scripts everywhere. I think solution with stop walking.

Last edited 10 days ago by aquadran (previous) (diff)

comment:4 by aquadran, 2 weeks ago

about "ImSetParam: getting name from number is not supported!" it's related to imuse, but maybe gracefully warning instead exit with error

comment:5 by aquadran, 2 weeks ago

Owner: set to aquadran
Resolution: assigned
Status: newpending

comment:6 by aquadran, 10 days ago

Owner: aquadran removed
Resolution: assignedpending
Note: See TracTickets for help on using tickets.