Shape destructor should be virtual

[axjxwright]

Non virtual destructors in base classes results in undefined behavior. ~Shape() should be marked as virtual to make the destruction of derived types well defined