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

Blind fix of delta operator #2863

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Blind fix of delta operator #2863

merged 1 commit into from
Nov 17, 2023

Conversation

Vurv78
Copy link
Contributor

@Vurv78 Vurv78 commented Nov 16, 2023

  • Fixed oversight with delta as I was trying to move the functionality from the parser to the compiler, which made using a delta variable before the compiler saw the $ operator being used treat it as a normal variable.
  • Fixed delta variables across includes which wouldn't be tagged as delta variables (I think this might've been an issue even before the compiler rewrite, I just happened to think of it.)
#include "Include_me"

Var = 0

printDelta()
@name Include me
@persist Var

Var = 5

function printDelta() {
    print($Var)    
}

So this code doesn't error now.

@CheezusChrust

@Vurv78 Vurv78 merged commit 52f823f into wiremod:master Nov 17, 2023
1 check failed
@Vurv78 Vurv78 deleted the delta3 branch November 17, 2023 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant