On 3 January 2012 18:07, Chase Douglas wrote:
> I rebuilt the library with this
> change to gcc and ran the test suite. All passed normally.

That's what I needed to know, your original mail didn't say anything
about running the test suite.

Looking at the patch again, why have you added destructors in
shared_ptr_base.h?  That part of the patch is not backported from the
PR 50500 fix, is it needed for Clang++?  If so, and G++ doesn't catch
the errors, that would imply the fix for 50500 was incomplete and so
should be reopened.

The ChangeLog is in the wrong format:

> PR c++/50500
> * Explicitly define default copy constructor for std::shared_ptr

This doesn't name the affected files.

Reply via email to