#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 |
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 , 20 years ago
| Summary: | Speech/Lip animation out of sync → FOTAQ: Speech/Lip animation out of sync |
|---|
comment:2 by , 20 years ago
comment:3 by , 20 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:5 by , 20 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 , 20 years ago
| Owner: | set to |
|---|
comment:7 by , 20 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 , 20 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 , 20 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
'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. :)