SubscriberInterface virtual destructor missing

[nfz]

Usually I wait a few days after cvs gets updated before I post about errors in cvs. Decided to be different this time:).

The base class SubscriberInterface does not have a virtual destructor so when 'delete d_si' in the template class SubscriberTemplate is executed, it only calls the base class's destructor and not those of the subclasses which results in leaks and exceptions in debug mode with the Ogre memory manager.

I added a virtual destructor to the SubscriberInterface class:

Code: Select all

class CEGUIBASE_API SubscriberInterface
{
public:
    virtual bool operator()(const EventArgs& args) const = 0;
    virtual ~SubscriberInterface() {}
};



[CrazyEddie]

Thanks, I actually thought about virtual dtors for the new events stuff in bed an hour ago I'll get this sorted later today.

CE.