Opened 19 years ago

Closed 19 years ago

#1965 closed defect (fixed)

MONKEY: French CD version incorrectly identified as monkey1

Reported by: SF/jellby Owned by: fingolfin
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Monkey Island 1


I have a Monkey Island CD which includes versions in several languages. When I add the French version with the GUI, it is identified as "monkey1, alt version", even though the files are named MONKEY.000 and MONKEY.001. If I try to start it, it complains with:

WARNING: md5_file couldn't open 'monkey1.000'! Unknown MD5 (1000000068cf980888ebffbf71a82d40)! Please report the details (language, platform, etc.) of this game to the ScummVM team

...but changing the gameid to "monkey" works fine.

ScummVM 0.8.0CVS (Mar 19 2005 22:12:02) Features compiled in: Vorbis FLAC MP3 ALSA zLib MPEG2

Ticket imported from: #1167146. Ticket imported from: bugs/1965.

Change History (5)

comment:1 by cyxx, 19 years ago

This one is very similar to 1009344, except that it's the other way around. We have the same md5 for 2 different targets. In the case of BR1009344, bsearch was returning the md5 for the 'monkey' target, it seems like on jellby system, bsearch now returns the md5 for 'monkey1'. Don't know what is the best solution (there are no guaranties about the returned value of bsearch in case of 'equal' elements, right ?)... Maybe add another workaround (the other way around) like you did, fingolfin, for the previous bug report ? Or turn the bsearch call into a simple, less efficient, linear loop in which we check the target is the expected one ?

Besides, if we add another workaround, what about modifying the md5table tool to emit a warning if there are duplicates md5s (it would prevent future bugs/problems like this) ?

comment:2 by fingolfin, 19 years ago

Summary: French CD version incorrectly identified as monkey1MONKEY: French CD version incorrectly identified as monkey1

comment:3 by fingolfin, 19 years ago

A warning in the md5tool would be a good idea in any case.

For the future, we want to get completly rid of the monkey/monkey1 distinction anyway, which will solve this problem automatically.

comment:4 by fingolfin, 19 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:5 by fingolfin, 19 years ago

I added monkey / monkey1 to the substResFileNameTable, it should work fine either way now. At least it does in my tests.

Note: See TracTickets for help on using tickets.