What it says on the tin. No exteranal functional changes, just refactoring.
Crate-internal test_construct function has been removed, as the associated test didn't seem to catch any errors. It has been rewritten to use a fuzzing-style property-based testing.
The purpose of this PR is to allow more complex instruction definitions in the future. This should allow moving some validity checks to instruction parsing level, and to improve type safety.
Checklist
[x] Breaking changes are clearly marked as such in the PR description and changelog (None!!)
[x] New behavior is reflected in tests (No new behavior!)
Before requesting review
[x] I have reviewed the code myself
[ ] I have created follow-up issues caused by this PR and linked them here
What it says on the tin. No exteranal functional changes, just refactoring.
Crate-internal
test_constructfunction has been removed, as the associated test didn't seem to catch any errors. It has been rewritten to use a fuzzing-style property-based testing.The purpose of this PR is to allow more complex instruction definitions in the future. This should allow moving some validity checks to instruction parsing level, and to improve type safety.
Checklist
Before requesting review