tree 623663557077c18952f3b1ffdc072f52cd1c3178
parent 019362fefcdca39a46d1a831ed46266c24b9a273
author Brian Silverman <bsilver16384@gmail.com> 1434499008 -0700
committer Aliaksey Kandratsenka <alk@tut.by> 1434509936 -0700

Add empty virtual destructor to class with virtual methods.

Clang 3.5 has a warning about deleting objects with virtual methods
through non-virtual destructors which was triggered. I'm not sure
whether this actually creates any undefined or otherwise incorrect
behavior, but it seems like a good thing to fix regardless.

Example compiler warning:
third_party/gperftools/src/tests/profile-handler_unittest.cc:282:5: error:
  delete called on '(anonymous namespace)::BusyThread' that has virtual
  functions but non-virtual destructor [-Wdelete-non-virtual-dtor]
    delete busy_worker_;
    ^