-
Notifications
You must be signed in to change notification settings - Fork 4
Remove arcwrapper and associated logic. #359
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
Conversation
|
@jonasbardino I've attempted to carefully perform the removal here, incluing any blocks that conditionally ventured into ARC related things. The only area I'm not clear on how to handle is |
Thanks, the |
1553266 to
49fee3d
Compare
49fee3d to
182c8ea
Compare
|
@jonasbardino have update this to the tip of next via the GitHub UI. Was there anything else that you’d wanted here? Otherwise reckon it’ll be nice to move this hunk of unused stuff out of the way. |
jonasbardino
left a comment
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.
Looks good, thanks. As it's only specific removals I suppose it's safe to merge.
Based on a response to a small cleanup attempt in acwrapper (#350 (comment)) it seems it is better to remove this code and its callsites. Do so here.