Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Special Handling Opcodes #21

Closed
OrionSR opened this issue Feb 12, 2020 · 1 comment
Closed

Special Handling Opcodes #21

OrionSR opened this issue Feb 12, 2020 · 1 comment
Labels
priority:p2 Medium priority scope:types type:feature New feature or request
Milestone

Comments

@OrionSR
Copy link

OrionSR commented Feb 12, 2020

The current issue is that special handling opcodes ignore command names and comments in sascm.ini and always decompile to text that isn't useful.
05B6 1 // CURRENT_WANTED_LIST = instead of 05B5: save_string_to_debug_file 1 // special handling

My other comment on special handling is a rather vague idea inspired by stumbling across an unusual opcode and not being able to resolve the matter without an update to Sanny. Can the 128 byte data blocks be typed (k%) so an opcode could be defined similar to:

0A68=3,write_log_float %1d% %2d% %3d% case %4k%

@OrionSR
Copy link
Author

OrionSR commented Feb 16, 2020

Another concern about special handling opcodes that goes more to future planning than an actual problem: I'm considering a plan to reprocess all opcodes.txt from all games at once by using an artificial opcode number and straightening out the actual opcode with another process. I expect the plan to work if I tip-toe around the special handling opcode numbers but it complicates the process.

So, as you plan other strategies for managing special handling opcodes, please consider something that doesn't tie directly to a specific opcode number. Let this be purely a function of the SCM.ini definition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:p2 Medium priority scope:types type:feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants