Boost C++ Libraries: Ticket #6943: gcc 4.7 issue in combination with intrusive_ptr_add_ref and intrusive_ptr_release (boost_1_49_0/boost/intrusive_ptr.hpp - boost_1_49_0/boost/smart_ptr/intrusive_ptr.hpp )

safety0ff.bugz@… — Fri, 15 Jun 2012 17:12:05 GMT

I had the same error/issue. My code had the form:

// Header 1
class SomeClass;
namespace boost
{
	void intrusive_ptr_add_ref(SomeClass* psRPFT);
	void intrusive_ptr_release(SomeClass* psRPFT);
};
class SomeClass
{
// stuff, including a private variable called reference_count
	friend void ::boost::intrusive_ptr_add_ref(SomeClass* psSC);
	friend void ::boost::intrusive_ptr_release(SomeClass* psSC);
};
namespace boost
{
	inline void intrusive_ptr_add_ref(SomeClass* psSC)
	{
		++(psSC->reference_count);
	}
	inline void intrusive_ptr_release(SomeClass* psSC)
	{
		--(psSC->reference_count);
	}
};
// Header 2
// include Header 1
using boost::intrusive_ptr;
// Source file 2
// Include header 2 and do something that invokes the copy constructor of intrusive_ptr

My solution was to change:

using boost::intrusive_ptr;

using boost::intrusive_ptr;
using boost::intrusive_ptr_add_ref;
using boost::intrusive_ptr_release;

It this was slightly unexpected for me.
It compiled without the change on gcc 4.5.3 but would not on clang 3.1 and gcc 4.7.0

Not sure if it is a bug (would like clarification.)

safety0ff.bugz@… — Fri, 15 Jun 2012 17:13:34 GMT

psRPFT should be psSC in previous comment.

owner, component changed

Ion Gaztañaga — Sun, 17 Jun 2012 06:34:42 GMT

owner changed from Ion Gaztañaga to Peter Dimov
component intrusive → smart_ptr

Changing componente as intrusive_ptr belongs to smart_ptr. I don't think it's a bug, the standard requires a previous declaration. You should include boost/intrusive_ptr.hpp before calling boost::intrusive_ptr_add_ref.

attachment set

safety0ff.bugz@… — Sun, 17 Jun 2012 07:00:12 GMT

attachment → intrusive.cpp

Reduced example of the issue

Peter Dimov — Sun, 17 Jun 2012 10:37:55 GMT

You should define intrusive_ptr_add_ref and intrusive_ptr_release in the namespace of the class, not in namespace boost.

safety0ff.bugz@… — Sun, 17 Jun 2012 17:35:36 GMT

This can be closed (that article contains bad information.)

Thanks for clearing that up.

status changed; resolution set

Peter Dimov — Wed, 31 Oct 2012 20:06:54 GMT

status new → closed
resolution → invalid