If you make a cl**** virtual (having at least one virtual function), you should also have a virtual destructor.
Here is a patch that does just this :
=== vehicle/overtaker.h
==================================================================
--- vehicle/overtaker.h (revision 131)
+++ vehicle/overtaker.h (local)
@@ -56,6 +56,9 @@
// since citycars and convois can react quite different
virtual bool can_overtake(overtaker_t *other_overtaker, int other_speed, int steps_other, int diagonal_length) = 0;
+
+ // Virtual destructor since there is at least one virtual function
+ virtual ~overtaker_t() { /* do nothing*/ };
};
#endif
Very nice patch btw.