[Neuro Toxin]

Nice. Looks bloody expensive for those who run massive plugin counts.

Was thinking of adding something like: Dynamic.DisposePluginObjects() which a plugin could call on OnPluginEnd to clean it's trash.

Registering a library is a much cleaner fix.

OnPluginUnload(Handle plugin) would be the best.

[asherkin]

Originally Posted by Neuro Toxin You could also .Dispose your Dynamic objects OnPluginEnd

This is the sane and correct thing to do, and why OnPluginEnd exists.

[Peace-Maker]

Would still error out if the plugin calls SetFailState after creating some dynamic objects

[Neuro Toxin]

I think a failed state is ok as GetPluginStatus has a status for this and the handle isn't yet destroyed until it's unloaded.

My issue GetPluginStatus errors on a plugin handle that has been completely unloaded.

[KyleS]

Originally Posted by Peace-Maker Would still error out if the plugin calls SetFailState after creating some dynamic objects

IForwardSys is your friend here. If you create a private forward in pawn, register the forward, then GetFunctionCount() you get a super cheap, protected check. If there's 0, you know your friend, has unloaded. This is what I did for my pawnhooks ext.

[Neuro Toxin]

Do the plugins have to implement the forward for GetFunctionCount to count them?

[KyleS]

Originally Posted by Neuro Toxin Do the plugins have to implement the forward for GetFunctionCount to count them?

No. You create a private forward, AddToForward, and you're done. You have a private forward handle to the plugin that you can use to call their functions. From the parent, you can then GetFunctionCount them to see if they're still loaded before calling their functions. It's a round-the-bout way, but it got me there years ago when I had a similar problem that needed solving.

[Neuro Toxin]

Oh.

So I could make my plugin reference (dynamic.OwnerPlugin) a forward handle and add the plugin to it and if the count is 0 the plugin has unloaded.

[asherkin]

Originally Posted by Neuro Toxin Oh. So I could make my plugin reference (dynamic.OwnerPlugin) a forward handle and add the plugin to it and if the count is 0 the plugin has unloaded.

Yes.

[Neuro Toxin]

The only thing I dont like about this is the use of an additional Handle.

SM allows approx 32k Handles. Dynamic uses 2 per object (lookup Trie and data ArrayList). Adding another Handle really cuts into the max objects that can be created by plugins.

Edit:

I could probably cache one Handle per plugin to avoid one new additional Handle for each .OwnerPlugin