Opened 18 years ago

Closed 17 years ago

Last modified 10 years ago

#229 closed defect (fixed)

You don't load the correct files?

Reported by: (none) Owned by: SF/ender
Priority: low Component: Engine: AGOS
Keywords: Cc:
Game: Simon the Sorcerer 1


ScummVM will crash loading the files. It
cannot find the files named in simon.cpp.

Simon.cpp tries uppercase, lower case attempts
to open a file before giving up. My simon
files (on a Simon 1 Talkie CD for DOS and
Windows 95) are lower case with a trailing

I am running Linux.

I have modified Simon.cpp from the 20020429
snapshot to fix some of this. I can get Simon1
loading and working fine, with one exception -
It tries to load a file I do not possess.

Sometimes it crashes with a segfault, but
fine if I skip the intro.

I have attached the directory listing of the
files on my CD.

Here are the changes to the cpp file

static const GameSpecificSettings
simon1_settings = {
1576/4, /* TABLE_INDEX_BASE */
1460/4, /* TEXT_INDEX_BASE */
1700/4, /* NUM_GAME_OFFSETS */
1000000, /* VGA_MEM_SIZE */
50000, /* TABLES_MEM_SIZE */
1316/4, /* MUSIC_INDEX_BASE */
//"SIMON.GME", /* gme_filename */
//"SIMON.WAV", /* wav_filename */
//"GAMEPC", /*
gamepc_filename */

// In the function fopen_maybe_lowercase add
at // end
/* if that fails, convert the filename into
lowercase and retry */

do *e = tolower(*e); while(*e++);

in = fopen(buf, "rb");
if (in ) return in;

// And add a . if needed
e = strchr(buf,0); // find end?
printf("MOD IS %s", buf);

return fopen(buf, "rb");

20020506 snapshot has same problem.

Ticket imported from: #552987. Ticket imported from: bugs/229.

Change History (7)

comment:1 by SF/ender, 18 years ago

Assigning to strigeus, as the simon code is his and the
rest of us don't understand a word of it. :)

Lowering priority as it's not a SCUMM bug.

comment:2 by SF/ender, 18 years ago

Owner: set to SF/strigeus
Priority: normallow

comment:3 by SF/ender, 17 years ago

Owner: changed from SF/strigeus to SF/ender
Resolution: fixed
Status: newclosed

comment:4 by SF/ender, 17 years ago

Fixed in latest CVS, as ludde is a nit who doesn't want to
maintain his own code :P

comment:5 by Kirben, 17 years ago

Component: Engine: AGOS
Game: Simon the Sorcerer 1
Owner: SF/ender removed
Status: closednew

comment:6 by Kirben, 17 years ago

Component: Engine: AGOS
Game: Simon the Sorcerer 1
Status: newclosed

comment:7 by Kirben, 10 years ago

Owner: set to SF/ender
Note: See TracTickets for help on using tickets.