#6417 DREAMWEB: Intro speech truncated in German Version

Dreamweb
open
digitall
5
2014-01-09
2013-10-04
Max
No

ScummVM version: 1.6.0 9 Juli 2013 (11:25:59)
Language: German CD
Version of game: Talkie cd version (Download from ScummVM website)
Plattform: Windows 7 64 bit
in the intro from dreamweb (at picture) It comes to a termination from the speech when the red man claps.

Discussion

  • Max

    Max - 2013-10-04
     
  • digitall

    digitall - 2013-10-05

    Hmm.. Replicated with the latest Git development master of ScummVM (1.7.0git) and the German version... The speech phrase as the keeper raises both his arms to produce the ball which wakes Ryan is clipped... truncated. This doesn't happen in the English UK version... but the keeper doesn't appear to be speaking at this point, so not sure why he is in the German version. The last phrase should be "The time has come and I shall awaken him." after the phrase "Yes. It has grown strong and he is stirring."

    frankenbaum: I don't speak German... are both equivalent phrases present?Or has the second one been clipped off...

    Running "scummvm -d 1" in the buggy case, this shows:
    stopSound(1)
    loadSpeech(T82T0017.RAW)
    playChannel1(index:62)
    playSound(channel:1, id:62, loops:1)
    cancelCh1()
    stopSound(1)
    loadSpeech(T82T0018.RAW)
    playChannel1(index:62)
    playSound(channel:1, id:62, loops:1)
    playChannel1(index:16)
    playSound(channel:1, id:16, loops:1)
    playChannel1(index:16)
    playSound(channel:1, id:16, loops:1)

    I think what is happening is that the ball animation sound is causing the speech playback to be aborted early... The original engine design relied on a number of "implicit" delays and we may need to add another delay or interlock to stop this happening.

    frankenbaum: Could you also try running this under the original interpreter in DOSBox (the freeware release includes the original DOS binary installer) and see if this occurs there as well... so we can check if this was a bug in the original German localisation or if this is a regression... Thanks.

     
  • digitall

    digitall - 2013-10-05
    • assigned_to: nobody --> tdhs
    • milestone: --> Dreamweb
    • labels: --> SFX/Talkie
     
  • digitall

    digitall - 2013-10-05
    • summary: Dreamweb (German), termination from speech in intro --> DREAMWEB: Intro speech truncated in German Version
     
  • Max

    Max - 2013-10-05

    hi tdhs! :)
    yes, that's right.
    I tested the german freeware version and my orignal CD and i found out, that probably in the scummvm version the entire part to late worn out. because in fact he spoke all sooner. also in the german version the keeper doesn't appear to be speaking when he both his arms to produce the ball which wakes Ryan.

    The german text..

    ...in the german scummvm version he says: yes. It has grown strong and he is stirring. his ..... [truncated]

    ...in orginal (dosbox): yes. It has grown strong and he is stirring. his ... name is ryan. The time has come and I shall awaken
    him.

    *man lifts his hands in the air*

     
  • digitall

    digitall - 2013-10-09

    frankenbaum: Hmm... Can you try testing with v1.5.0 and then if you are capable of compiling ScummVM yourself, doing a git bisection to track down the regression point... as I suspect this may have worked at some point in the development. I will try to do the same, but it may take me some time, especially as I don't speak German so trickier to spot the truncation of speech...

     
  • digitall

    digitall - 2013-10-09

    frankenbaum: Don't worry... I have done this work myself. The bug occurs in v1.5.0 and have run git bisection to locate the regressive commit. This is identified as:
    commit 1193792310815b5f83fcf4fa5f018c4a41f4131a
    Author: Willem Jan Palenstijn <wjp@usecode.org>
    Date: Tue Dec 6 19:23:38 2011 +0100

    DREAMWEB: Fix regressions in 'monks2text'

    However, the audio prior to this commit is also glitching in the intro (repeated phrases) but not truncating.

    The cause of this appears from the content of the commit to be a faulty conversion of the "monks2text" function to C++ or a latent timing bug exposed by the conversion. The conversion was actually done by:
    commit d5b220220f61a16c0eaaa9ca88e3e0b4d39d1d33

    Filippos Karapetis <md5@scummvm.org> 2011-12-06 10:35:47

    DREAMWEB: 'intro1text', 'intro2text', 'intro3text', 'monks2text' ported to C++. textForMonk() has been converted to C++, too.

    Thus commit 7d9d53d2badfdf124fa67360599087f1e5fc06e2 prior to this being the last "good" commit and I have tested this as thus.

    Hopefully with this information, we can fix this issue.

     
  • digitall

    digitall - 2014-01-09

    Have done some further investigation on this. The issue is in the monks2text function conversion, which is still not correct even after wjp's 1193792310815b5f83fcf4fa5f018c4a41f4131a corrections.

    The original ASM for this function has three different versions for German CD, German Floppy and All Other Builds. The dreamgen.cpp output for tasm-recover only converts one version of this (All Other) and thus wjp needed to manually translate and add support for the other two cases.

    Either he made a minor mistake in some of the logic or the parameters passed to setupTimedTemp() calls in the German case, or the original ASM is incorrect for these cases...

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks