Opened 15 years ago

Closed 15 years ago

Last modified 10 years ago

#1401 closed defect (fixed)

BS1: Cutscenes don't respond

Reported by: Kirben Owned by: eriktorbjorn
Priority: normal Component: Engine: Sword1
Keywords: Cc:
Game: Broken Sword 1

Description

Latest ScummVM cvs version.
English version of Broken Sword 1
Compiled under mingw with GCC 3.2.3 and running under
Windows XP.

ScummVM doesn't respond to keyboard commands when
cutscenes are playing, ie alt enter.
If I switch to another program and back to ScummVM, it
seems to lock up. The audio continues in background but
display is no longer updated.

Shouldn't the delays for decodeFrame() in
sword1/animation.cpp be going through delay() instead
of system backend directly ?

Ticket imported from: #881112. Ticket imported from: bugs/1401.

Change History (6)

comment:1 Changed 15 years ago by SF/khalek

Component: Engine: Sword1
Game: Broken Sword 1

comment:2 Changed 15 years ago by eriktorbjorn

Owner: set to SF/ender

comment:3 Changed 15 years ago by eriktorbjorn

The patch I attached to bug #880484 has a minimal fix to
this since it calls poll_event() as well as delay_msecs(),
but that's just because I wanted to keep the patch minimal.
(The delay() function is private.)

You're right that if all we do is call delay_msecs() ScummVM
won't respond to standard events. In the SDL backend that
includes the SDL_VIDEOEXPOSE event, which is probably why
ScummVM appears to lock up. It doesn't realize that it has
to redraw the game window.

Assigning to Ender, since he's the one doing cutscene work.
I think.

comment:4 Changed 15 years ago by eriktorbjorn

The patch I attached to bug #880484 has a minimal fix to
this since it calls poll_event() as well as delay_msecs(),
but that's just because I wanted to keep the patch minimal.
(The delay() function is private.)

You're right that if all we do is call delay_msecs() ScummVM
won't respond to standard events. In the SDL backend that
includes the SDL_VIDEOEXPOSE event, which is probably why
ScummVM appears to lock up. It doesn't realize that it has
to redraw the game window.

Assigning to Ender, since he's the one doing cutscene work.
I think.

comment:5 Changed 15 years ago by Kirben

Owner: SF/ender deleted
Resolution: fixed
Status: newclosed

comment:6 Changed 10 years ago by Kirben

Owner: set to eriktorbjorn
Note: See TracTickets for help on using tickets.