Opened 14 years ago

Closed 13 years ago

#5386 closed defect (fixed)

AGI LSL1: TAB stops working after restart

Reported by: eriktorbjorn Owned by: bluegr
Priority: high Component: Engine: AGI
Version: Keywords:
Cc: Game: Leisure Suit Larry 1

Description

Current 1.2.0 branch English version of the game (1.00 1987-06-01, 5.25"/3.5"/DOS/English, according to the detector)

This one's easy to reproduce: Start the game, andswer the questions, and type "restart" (or walk into a dark alley and let the game restart on its own). The TAB key no longer brings up the inventory. It seems the "game controllers" are reset on restart, but this game never recreates them. (I tried Space Quest 2, and the bug didn't happen there.)

Ticket imported from: #3074570. Ticket imported from: bugs/5386.

Attachments (1)

agi-controllers.diff (2.9 KB ) - added by eriktorbjorn 14 years ago.
Possible patch against 1.2.0 branch

Download all attachments as: .zip

Change History (10)

comment:1 by eriktorbjorn, 14 years ago

I forgot to mention: The bug does not happen with the original interpreter in DOSBox.

comment:2 by eriktorbjorn, 14 years ago

This bug does not seem to happen in NAGI. (http://github.com/sonneveld/nagi)

comment:3 by eriktorbjorn, 14 years ago

Unlike ScummVM, NAGI doesn't appear to ever clear the controllers list, except in nagi_init(), which appears to only be called once. If cmd_set_key() fails to find a free slot, it just silently ignores it.

ScummVM's cmd_set_key() also ignores attempts to overflow the list, but prints a warning.

by eriktorbjorn, 14 years ago

Attachment: agi-controllers.diff added

Possible patch against 1.2.0 branch

comment:4 by eriktorbjorn, 14 years ago

Assuming I understood the problem correctly, the patch I just attached should fix the problem. It doesn't work exactly like how I think NAGI works, but...

comment:5 by sev-, 14 years ago

I'll review the patch.

comment:6 by sev-, 14 years ago

Owner: set to sev-
Priority: normalhigh

comment:7 by eriktorbjorn, 13 years ago

Any news about this?

comment:8 by bluegr, 13 years ago

Fixed in r81fdf2c, by applying eriktorbjorn's patch and checking against NAGI

comment:9 by bluegr, 13 years ago

Owner: changed from sev- to bluegr
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.