Opened 16 years ago

Closed 16 years ago

Last modified 2 years ago

#8389 closed patch

LOOMCD: Workaround for typo in original game

Reported by: eriktorbjorn Owned by: eriktorbjorn
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Loom

Description

This patches works around a typo in the CD version of Loom, where Chaos introduces herself to Mandible as "Choas".

There are two reasons why I haven't just commited this without asking. First of all, I don't know if there were any non-English versions of LoomCD, and secondly I don't know if we want to do this kind of fixes.

Ticket imported from: #1093830. Ticket imported from: patches/494.

Attachments (2)

loom-typo.diff (735 bytes ) - added by eriktorbjorn 16 years ago.
Patch against a December 30 CVS snapshot
loomcd.s10 (10.5 KB ) - added by eriktorbjorn 16 years ago.
Savegame before Chaos appears

Download all attachments as: .zip

Change History (6)

by eriktorbjorn, 16 years ago

Attachment: loom-typo.diff added

Patch against a December 30 CVS snapshot

by eriktorbjorn, 16 years ago

Attachment: loomcd.s10 added

Savegame before Chaos appears

comment:1 by fingolfin, 16 years ago

Well, we gotta be certain before we add stuff which changes game play, but in this case, the fix is simple, and the issue it fixes is rather, uhmm, embarrassing, so I don't mind it at all.

As for the implementation: I am pretty sure there were no non-english versions of LoomCD. And even if there is, you are doing a memcmp, so no harm done (I'd use a strcmp, though, it's clearer to understand I think).

One request: Please add a "WORKAROUND" at the start of your comment, like we use in other places (to make it clear that this is really a well considered workaround for a problem; not just a random hack)

comment:2 by eriktorbjorn, 16 years ago

Well, normally we can't use strcmp() since the string can contain embedded zeroes - at least that's my understanding - but I guess that's not a problem in this particular case.

I'm applying the patch, with your proposed changes, as soon as I've verified that it still works.

comment:3 by eriktorbjorn, 16 years ago

Owner: set to eriktorbjorn
Status: newclosed

comment:4 by digitall, 2 years ago

Component: Engine: SCUMM
Game: Loom
Note: See TracTickets for help on using tickets.