#6014 SIMON1DOS French: subtitle glitch in introduction

closed-fixed
kirben
5
2012-04-01
2012-03-29
misterhands
No

SCUMMVM 1.4.1, Simon1 CD/DOS/French

Not really a bug, more a minor glitch, but I thought I'd report it anyway.
In the intro during the goblins_cooking_poor_simon scene, when the goblin
pushes Simon in the pot and says "down food", the French subtitle displayed is :
"Plus bas,

ourriture."
Notice the added blank line in the middle, and the missing first letter "n" in the word nourriture.
Interestingly, the game crashes at that very spot in dosbox, which makes scummVM even more essential I may add.
So it looks like the added blank line and missing letter are a byproduct of a scummVM hack/fix to address the crash.
See bug ID: 770838, it seems to be related.

Anyway, a quick look in SIMON.GME reveals that the missing "n" is there so I assume it should be fixable without to much sweat.
As for the added line, I can live with it, but I'll let you C++ maestros decide if something can or should be done about it.

Thanks!

Discussion

  • digitall

    digitall - 2012-03-29
    • assigned_to: nobody --> kirben
     
  • digitall

    digitall - 2012-03-29

    Hmm... Looks like a regression since the fix in bug #770838 "SIMON1DOS: Fatal crashes with French version":
    https://sourceforge.net/tracker/?func=detail&aid=770838&group_id=37116&atid=418820

    Since this was 2003-07-14 03:18, that means the fix was in 0.4.0..

    misterhands: Could you try this with some older versions i.e. v1.4.0, v1.3.1, v1.2.1, v1.1.1, v1.0.0 etc. to try to see if this was a recent regression? Could you also be clear what platform you are using i.e. Win32, Linux, MacOSX?

     
  • misterhands

    misterhands - 2012-03-30

    Ok I've tried a bunch of older versions from 1.4 to 0.4 and basically there is no difference,
    other than showcasing the evolution of scummvm !
    It looks like the missing letter and added line have been there since the fix in 2003.
    I use win32.

     
  • digitall

    digitall - 2012-03-31

    misterhands: Thanks for this update.
    Looking at this, the correction for the crash with the Simon1DOS French version was added in commit
    4ae380102a533a8a332e46a4daea2a273f3930a1

    Fix for Bug [770838] SIMON1DOS: Fatal crashes with French version

    Fix variable underflow for large subtitle strings with no
    discernable whitespace. This fixes crashes for French subtitles
    that do, in fact, have whitespace but perhaps not using the
    same ASCII character that we scan for. Anyway, the text will
    display, albeit with an extra blank line, and ScummVM no longer
    blows up.

    svn-id: r9009

    This corrected some code in the function:
    void SimonEngine::talk_with_text(uint num_1, uint num_2, const char *string_ptr,...

     
  • digitall

    digitall - 2012-03-31

    This function no longer exists in that file or with that name.

    However, I think the relevant code has been moved to engines/agos/string.cpp and is now within the function "void AGOSEngine::printScreenText(uint vgaSpriteId, uint color, const char *string, int16 x, int16 y, int16 width)"

    Hopefully this should help the AGOS developers look at a possible fix for this issue...

     
  • Johannes Schickel

    • summary: SIMON1DOS French : subtitle glitch in introduction --> SIMON1DOS French: subtitle glitch in introduction
     
  • digitall

    digitall - 2012-04-01

    Hmm... commit e457a3f2513529930d16636155243dfc125035bd :
    " Add workaround for bug #1538873 - SIMON1 (French): Text wrongly displayed.

    svn-id: r42474"
    may also be relevant to this...

     
  • kirben

    kirben - 2012-04-01
    • status: open --> closed-fixed
     
  • kirben

    kirben - 2012-04-01

    Fixed in ScummVM Git, use the next daily snapshot of ScummVM GiT.

    The problem was due to the French version failing to update the width, for this particular string. I added a work around to set the correct width.

    Please report any other locations you notice this problem, to this bug report, so they can be worked around too.

     
  • misterhands

    misterhands - 2012-04-02

    Sorry this is not fixed for me...
    This is with the daily snapshot 1.5.0git2811-gb886bc0 (Apr 2 2012 11:00:30)

     
  • kirben

    kirben - 2012-04-02

    Fixed in ScummVM Git, use the current Windows snapshot of ScummVM GiT.

    The string ids were different in the Floppy Disk and CD version, so I had to expand the bug fix.

     
  • misterhands

    misterhands - 2012-04-02

    It works ! thanks !

    I did a play through of Simon 1 recently and I did not notice other occurrences of this bug.
    I did however notice other bugs, I'll report them later.