Opened 15 years ago

Closed 15 years ago

Last modified 6 years ago

#9092 closed patch (wontfix)

descumm: output verb metadata for SCUMM V3-4

Reported by: SF/jestar_jokin Owned by: fingolfin
Priority: normal Component: Tools
Version: Keywords:
Cc: Game:

Description

Outputs verb metadata for SCUMM V3-4 scripts ("OC" blocks).

The main reasoning for this is to output all information that was present in the original script block, so that third-party tools such as Scummbler can reconstruct the original script block using just the output from descumm.

Diff is based on revision 34337 of descumm-tool.cpp from SVN trunk.

Ticket imported from: #2870744. Ticket imported from: patches/1197.

Attachments (1)

descumm-tool.diff (1.1 KB ) - added by SF/jestar_jokin 15 years ago.

Download all attachments as: .zip

Change History (5)

by SF/jestar_jokin, 15 years ago

Attachment: descumm-tool.diff added

comment:1 by fingolfin, 15 years ago

Hm, I am really sceptical about this patch. Descumm is meant for disassembling SCUMM byte code; adding this functionality seems out of scope for it. It's only by pure laziness that this data is included in v3/v4 script dumps, too.

There once was a guy who worked on a general "SCUMM unpacker" tool, which could take apart a set of SCUMM resource files (creating many smaller files with the relevant data in them), and later re-assemble them. That would indeed be an interesting tool. But out of scope for descumm. Sadly, he stopped working on that stuff and vanished :/.

Anyway, the gist here is this: If you want to extract data from SCUMM files and re-assemble it later on, you should write a dedicate tool for it, not abuse descumm. As such, I am not very inclined to accept this patch... But maybe you have some strong arguments why this should be in descumm after all?

comment:2 by SF/jestar_jokin, 15 years ago

Not really, your points are perfectly valid, and on consideration this functionality is probably more suited to a general SCUMM unpacker. Feel free to close/reject this patch.

comment:3 by fingolfin, 15 years ago

Owner: set to fingolfin
Resolution: wontfix
Status: newclosed

comment:4 by digitall, 6 years ago

Component: Tools
Note: See TracTickets for help on using tickets.