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

[DECOHack] Global Labels for Filled State Sets #34

Open
MTrop opened this issue Jul 3, 2021 · 1 comment
Open

[DECOHack] Global Labels for Filled State Sets #34

MTrop opened this issue Jul 3, 2021 · 1 comment
Labels
enhancement Nice-to-haves and things that can actually help current behavior. low priority Issues labeled this are lower priority than regular bugs - rarer issues with workarounds.

Comments

@MTrop
Copy link
Owner

MTrop commented Jul 3, 2021

The state fill <INTEGER> blocks can fill arbitrary states, but if you have a large range of freed states, it makes little sense to care about the index, especially if it may be filled by the time this clause is reached in code.

Proposal:

auto state fill <IDENTIFIER>
{
  // state code...
}

The first index of the filled block will be assigned <IDENTIFIER>, which is checked for non-use. If not used yet, it is assigned and available to the rest of DECOHack from that point forward. Label references can now have a scope outside Things and Weapons.

But how to refer to these later? Maybe take a page from Issue #31:

auto state <IDENTIFIER>

For example:

A_RandomJump(auto state runcycle2, 128)
goto auto state resurrect

This will make the reference clear in Things and Weapons, to ensure label references do not get shadowed.

@MTrop MTrop added the enhancement Nice-to-haves and things that can actually help current behavior. label Jul 3, 2021
@MTrop MTrop added the low priority Issues labeled this are lower priority than regular bugs - rarer issues with workarounds. label Mar 7, 2022
@MTrop
Copy link
Owner Author

MTrop commented Mar 7, 2022

This may be obsoleted by some other plans, but I'll tag this with "low priority" as it is not a current concern.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Nice-to-haves and things that can actually help current behavior. low priority Issues labeled this are lower priority than regular bugs - rarer issues with workarounds.
Projects
None yet
Development

No branches or pull requests

1 participant