Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#1783 closed defect (fixed)

FOTAQ: Speech/Lip animation out of sync

Reported by: salty-horse Owned by: cyxx
Priority: normal Component: Engine: Queen
Keywords: Cc:
Game: Flight of the Amazon Queen

Description

All versions of ScummVM:
When in a close-up discussion with a character occurs,
such as the one with Faye in the airplane, her lips
always stop moving a second or so before the actual
speech sound ends.
It is also visible in the Ironstein cutscene at the
beginning of the game.
The original interpreter doesn't seem to have that
problem. I used the same talk speed in both (or more
accurately, the talk speed bar selector was graphically
in the same location).

To test, just jump to room 1.

Ticket imported from: #1034715. Ticket imported from: bugs/1783.

Change History (9)

comment:1 by fingolfin, 15 years ago

Summary: Speech/Lip animation out of syncFOTAQ: Speech/Lip animation out of sync

comment:2 by joostp, 15 years ago

'Fixing' this might require some substantial changes to the
Talk class, which we are currently investigating.

Since it's not a show-stopper; have patience. :)

comment:3 by cyxx, 15 years ago

I just committed some code in order to fix this "bug".

Now, the talk speed bar is only useful when the speech is
muted. When the speech is on, the mouth of the character is
closed as soon as the sfx is finished.

comment:4 by fingolfin, 15 years ago

What is the status of this item?

comment:5 by salty-horse, 15 years ago

cyx, your patch fixed the problem.
Regarding text speed, it would be nice if it worked like
SCUMM does: If speech is active, the text speed keeps the
text on screen and the speech animation running, regardless
of the sound. Minimal text speed means that the speech will
be in sync with the animation.

comment:6 by cyxx, 15 years ago

Owner: set to cyxx

comment:7 by cyxx, 15 years ago

In fact, I don't know what is the 'nicest' behaviour :
- your suggestion
- or, disabling the text speed scroll bar if sound speech is
checked

comment:8 by cyxx, 15 years ago

I prefer not modifying the talk code now (so we'll go with
the latter behaviour). If someone here isn't happy with it,
feel free to commit sthg else, send a patch or submit a RFE
; as I plan to close this entry since the bug described here
isn't 100% related to that configuration stuff...

Coming back to the original bug, I noticed, during my last
test run, that the Cutaway code was also handling some
talking stuff, so I updated that part too.

comment:9 by cyxx, 15 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.