AbleScript
/
ablescript
3
4
Fork 1
Code Issues Pull Requests Projects Releases 7 Wiki Activity

Changed the way how Melo-ing variables works #8

Merged
Ghost merged 3 commits from improvements/melo into master 2022-04-23 11:54:23 +00:00
Conversation 0 Commits 3 Files Changed 2
Ghost commented 2022-04-22 19:44:28 +00:00

So, Melo-ing variable in AbleScript is currently just setting a flag on it, keeping the value alive. And AbleScript currently has no way how do delete variables (except dropping the stack frame), so, I changed Variable to be an enum of Ref(ValueRef) or Melo. So, if the value is set to the Melo variant, ValueRef is dropped (and likely gets deallocated, as there is currently no way to copy references with variables). And if you Melo again, the variable will get erased from the top-most stackframe containing it.

So, Melo-ing variable in AbleScript is currently just setting a flag on it, keeping the value alive. And AbleScript currently has no way how do delete variables (except dropping the stack frame), so, I changed Variable to be an enum of Ref(ValueRef) or Melo. So, if the value is set to the Melo variant, ValueRef is dropped (and likely gets deallocated, as there is currently no way to copy references with variables). And if you Melo again, the variable will get erased from the top-most stackframe containing it.
Ghost added 3 commits 2022-04-22 19:44:29 +00:00
b0e8acd438 Reimplemented Melo 
- Variable made into enum of `ValueRef or Melo`
- Melo-ing variable causes dropping of the Rc, decreasing reference count
Ghost merged commit 45bb1ade84 into master 2022-04-23 11:54:23 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
panic

Program is panicking
  
bug

Something is not working   
duplicate

This issue or pull request already exists   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
question

More information is needed   
wontfix

This won't be fixed
No Label
panic
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AbleScript/ablescript#8
There is no content yet.
Delete Branch "improvements/melo"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?