On Fri, Nov 20, 2009 at 21:05, Evan Martin <e...@chromium.org> wrote:

> Your analysis is correct.  However, a virtual destructor is not needed
> in the case where you never delete through the Base*.  It turns out
> for our codebase that is very common (due to lots of "observer"-like
> patterns), so we decided to not rely on this compiler warning.  It's
> only caused horrible bugs once or twice!  :)
>

Is it worth to be more "tricky" at the expense of wasting time for
debugging? I hate it when after investigating some crazy bug it turns out to
be something that a compiler can detect.

-- 
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev

Reply via email to