This can be replaced with better direct C++ code.

Problems with this macro that this will solve:

• It confuses clang-format, which doesn't know that its VA_ARGS are C++ code.
• It currently does incomplete exception handling (missing boost::exception).
• It hides a catch(...). It's better for those to be visible, as they're important.
• It is used outside of destructors often for some reason.
• It does too much. Users cannot do anything else with the exception. It's gone when they return.

Replacing it with a function reportFailedDestructor.

Can remove the destructor_guard.h header and just put this function into assert_util.