#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)
Change History (5)
by , 15 years ago
| Attachment: | descumm-tool.diff added |
|---|
comment:1 by , 15 years ago
comment:2 by , 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 , 15 years ago
| Owner: | set to |
|---|---|
| Resolution: | → wontfix |
| Status: | new → closed |
comment:4 by , 5 years ago
| Component: | → Tools |
|---|
Note:
See TracTickets
for help on using tickets.
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?