Opened 19 years ago

Closed 19 years ago

Last modified 19 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
Version: Keywords:
Cc: Game: Flight of the Amazon Queen


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, 19 years ago

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

comment:2 by joostp, 19 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, 19 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, 19 years ago

What is the status of this item?

comment:5 by salty-horse, 19 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, 19 years ago

Owner: set to cyxx

comment:7 by cyxx, 19 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, 19 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, 19 years ago

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