Tgui::Gui has virtual methods but no virtual destructor

[texus]

Tgui::Gui has virtual methods but no virtual destructor

[texus]

But is it really needed to implement a virtual destructor? It is only needed when you are using a pointer to a derived class. And Gui isn't intended to be used as base class, neither should u use new and delete.

[Wizzard033]

The usage of new and delete with tgui::Gui in my circumstance comes from binding TGUI to a scripting language.
As I am not using C++ when I create and delete instances, using C++ scoping rules is out of the question.

[Wizzard033]

I suppose instead of using new/delete I could create a global container of GUI instances...
This is a nuisance though. My previous code worked and only broke as a result of updating TGUI.
The scripting language I use, Squirrel, was already made for plugging in new/delete or malloc/free in this scenario.
Doing anything different would be a hack...

Edit: Extra, unnecessary copying would be involved too in making the global container hack...

[texus]

Using new/delete is ok then in your situation, and you should be able to safely ignore this warning.
I'll either implement an empty virtual destructor, or mark the class as final. But I'll see about that later.

[Wizzard033]

Removing the virtual keyword from all members that have it is an option too...

[texus]

I'll look into this later. I'm currently writing something for LD, so I don't want to spend too much time on my gui right now.
And because it is just a warning I think it can be ignored for a few days.

[Wizzard033]

No problem
Thank you for the nearly immediate responses.