Opened 13 years ago

Closed 10 years ago

Last modified 10 years ago

#2780 closed defect (fixed)

SIMON1 (French): Text wrongly displayed

Reported by: SF/hibernatus Owned by: Kirben
Priority: high Component: Engine: AGOS
Keywords: original Cc:
Game: Simon the Sorcerer 1

Description

Look at the screenshot.
Line 3 (C'etait delicieux...) contains whitespaces at
the end which make it take 2 lines instead of one.

Ticket imported from: #1538873. Ticket imported from: bugs/2780.

Attachments (1)

Spaces.gif (45.6 KB) - added by SF/hibernatus 13 years ago.
Line 4 is wrongly displayed because of whitespaces at the end of line 3

Download all attachments as: .zip

Change History (22)

Changed 13 years ago by SF/hibernatus

Attachment: Spaces.gif added

Line 4 is wrongly displayed because of whitespaces at the end of line 3

comment:1 Changed 13 years ago by Kirben

Owner: set to Kirben

comment:2 Changed 13 years ago by Kirben

I think these text related bugs, occured in the original
French version too.

It would be helpful if you could try the game under dosbox,
to confirm if the issue occurs in the original game too.

You can use the saved game from ScummVM. in the original
game too. You will just need to unpack (They are gzip
compressed) and rename the filenames of the saved games.

comment:3 Changed 13 years ago by SF/hibernatus

Ah, you're right. Sorry.
It's strange, because it's only because of the trailing
whitespaces, so i thought the original engine would remove
them, but it doesn't.

comment:4 Changed 13 years ago by fingolfin

Status: newpending
Summary: Simon1: Text wrongly displayedSIMON1: Text wrongly displayed

comment:5 Changed 13 years ago by fingolfin

Can this be closed then?

comment:6 Changed 13 years ago by Kirben

Status: pendingnew
Summary: SIMON1: Text wrongly displayedSIMON1 (French): Text wrongly displayed

comment:7 Changed 13 years ago by Kirben

A work around might be possible, but I don't have the
French version of Simon the Sorcerer 1 myself.

comment:8 Changed 13 years ago by fingolfin

Owner: changed from Kirben to eriktorbjorn

comment:9 Changed 13 years ago by fingolfin

Couldn't you try to implement a workaround, then ask hibernatus to check
whether it works?

For example, one could just alway strip trailing whitespaces when printing
text. Or is that likely to cause regressions? If so, it shouldn't be too hard to
only perform this for specific texts... ?

I would try to do this, but I am not very familiar with the Simon/AGOS engine,
so it'd probably be done quicker if Kirben could try it. Or maybe Torbjörn is
interested in looking into this?

comment:10 Changed 13 years ago by eriktorbjorn

I don't know whether or not it'd cause regressions. I'm not
all that familiar with the AGOS engine.

I believe all those conversation messages go through
showMessageFormat(), but there are other things that do as well.

comment:11 Changed 13 years ago by SF/hibernatus

It could cause a regression if a translation uses that
"feature" to position sentences in front of numbers instead
of numbers in front of sentences.

For example:
1 Blah blah...
...blah?
2 Blah blah?

translated to:
1 Bla bla... bla?
<- blank line
2 Bla bla?

I've never seen that in the french version.

I don't see any other reason for a regression.

Here is a little tool which would help finding all the
concerned sentences in every translation.
http://hibernatus34.free.fr/simon/SimonTR.exe
(windows only)

comment:12 Changed 12 years ago by bluegr

Is it possible that this bug has been fixed with change #25807 ?

comment:13 Changed 12 years ago by Kirben

No, that change was unrelated.

comment:14 Changed 12 years ago by Kirben

Could you please try the dump all subroutines option, in the next snapshot of ScummVM SVN, and attach a file of the output.

Start ScummVM via command line, with output redirected to a file. ie:
scummvm simon1 > simon1fr.txt

Wait for the game to be displayed.
Press 'CTRL u' to dump all subroutines.
Wait a minute or two for output to complete.
Quit ScummVM.

Then just attached the file 'simon1fr.txt', to this bug report.

The output should make it much easier to attempt a work around, based on the specific strings where it occurs.

comment:15 Changed 12 years ago by SF/richiefs

Here are the requested data:
http://dl.free.fr/izbLY2oYy/simon1fr.txt

comment:16 Changed 11 years ago by sev-

What is the status of this item?

comment:17 Changed 10 years ago by sev-

This bug is nice to get fixed before the release. Raising priority for keeping the track.

comment:18 Changed 10 years ago by sev-

Owner: changed from eriktorbjorn to Kirben
Priority: normalhigh

comment:19 Changed 10 years ago by Kirben

Keywords: original added
Resolution: fixed
Status: newclosed

comment:20 Changed 10 years ago by Kirben

Fixed in ScummVM SVN, use the next daily snapshot of ScummVM SVN.

comment:21 Changed 10 years ago by Kirben

For future reference:
This problem effects the Polish version of Simon the Sorcerer 2 too.

Note: See TracTickets for help on using tickets.