2 Years Ago
Disable ToolRenderContext Commenting this out for this PR. Here's the issue with doing it like this. You're passing a struct down to addons, which can then be copied, which will copy the CToolRenderContext pointer. Which means you've lost control of it. That could lead to the situation where people are saving the struct somewhere and using it after the native pointer is dead. This is why where possible I've preferred the globals (Paint, Render) and protecting the pointer by nulling it when out of scope. This needs more thought on how it'd work for multiple views etc.