#229 closed defect (fixed)
You don't load the correct files?
Reported by: | (none) | Owned by: | SF/ender |
---|---|---|---|
Priority: | low | Component: | Engine: AGOS |
Version: | 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 , 23 years ago
comment:2 by , 23 years ago
Owner: | set to |
---|---|
Priority: | normal → low |
comment:3 by , 22 years ago
Owner: | changed from | to
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:4 by , 22 years ago
Fixed in latest CVS, as ludde is a nit who doesn't want to maintain his own code :P
comment:5 by , 22 years ago
Component: | Engine: AGOS |
---|---|
Game: | Simon the Sorcerer 1 |
Owner: | removed |
Status: | closed → new |
comment:6 by , 22 years ago
Component: | → Engine: AGOS |
---|---|
Game: | → Simon the Sorcerer 1 |
Status: | new → closed |
comment:7 by , 15 years ago
Owner: | set to |
---|
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.