Opened 18 years ago

Closed 18 years ago

Last modified 11 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

Description

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 period.

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. SFXXXX01.

Sometimes it crashes with a segfault, but works 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 = { 1, /* VGA_DELAY_BASE */ 1576/4, /* TABLE_INDEX_BASE */ 1460/4, /* TEXT_INDEX_BASE */ 1700/4, /* NUM_GAME_OFFSETS */ 64, /* NUM_VIDEO_OP_CODES */ 1000000, /* VGA_MEM_SIZE */ 50000, /* TABLES_MEM_SIZE */ 3624, /* NUM_VOICE_RESOURCES */ 1316/4, /* MUSIC_INDEX_BASE */ 0, /* SOUND_INDEX_BASE */ //"SIMON.GME", /* gme_filename */ //"SIMON.WAV", /* wav_filename */ //"GAMEPC", /* gamepc_filename */ "simon.gme", "simon.wav", "gamepc." };

// 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? strcpy(e,"."); 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, 18 years ago

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

comment:4 by SF/ender, 18 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, 18 years ago

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

comment:6 by Kirben, 18 years ago

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

comment:7 by Kirben, 11 years ago

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