-
Notifications
You must be signed in to change notification settings - Fork 73
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
More pedantic explanation of delegate context #95
More pedantic explanation of delegate context #95
Conversation
Depending of actual closure allocation, delegate context pointer may link to either enclosing function stack or heap copy.
a nested function includes a link to the enclosing context | ||
instead. However, D compiler may automatically make a copy of | ||
the context on the heap if it is necessary for memory safety - | ||
in that case delegate will link the that heap area. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the that -> to this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in that case -> in this case
@Dicebot FYI: if you make your changes on a branch of |
You can simply add my remote and push to my PR branch if you want to fix some grammar bits directly. It doesn't have to be a branch of Creating PR branches in main repo is too noisy, it should be reserved for bigger feature branches. |
Okay, well the advantage of a branch being here is that immediately all collaborates have access, but yes that works for me too. Thanks!
Argh .. you get an email whenever I create a new branch? :O |
What is the issue with travis?
|
Yeah that's on me (with the 1.0.2 changes this fix was introduced), but it's already fixed on the main repo, so I just restarted Travis. |
Depending of actual closure allocation, delegate
context pointer may link to either enclosing function stack
or heap copy.