OpenDreamProject / OpenDream

A project for running games made in the DM programming language
MIT License
200 stars 109 forks source link

CI should verify ProcDecoder is keeping up with opcode modifications #1879

Closed wixoaGit closed 1 month ago

wixoaGit commented 2 months ago

ProcDecoder is often forgotten when modifications are made to opcodes. This leads to a broken debugger and DMDisassembler.

A test to ensure DMDisassembler's test-all works on a SS13 codebase would be nice. An alternative would be to rewrite ProcDecoder to use OpcodeMetadataAttribute so it can work with most opcodes without needing explicit work.