 |
Forgot to set this as 'major' instead of 'minor'. |
|
|
...Hmmm. I think it's something to do with the recent Ammo scriptifying. |
|
|
Okay, updated to the latest with the health changes. It happens to those too. |
|
|
You have to call 'Destroy' not 'OnDestroy'. 'OnDestroy' is just a callback to report that the item is about to be destroyed but does nothing itself, hence the 'On...' in the name. For technical reasons it is not possible to make 'Destroy' itself virtual because that'd badly crash an engine shutdown. |
|
|
The original problem was solved, so it could be possible to try again to make Destroy virtual... |
|
|
I already solved the problem by making the cleanup function something that is being called by the actual Destroy method. This is absolutely necessary here, because Destroy can be called after the class table cleanup where a virtual scripted call will thoroughly crash the engine if not done properly and that means keeping code execution under tight control. The original implementation was something I never liked because it depended on everybody implementing it properly, if just one intermediate class forgets to call the super method big problems would ensue. This cannot happen anymore. |
- Anonymous
