diff --git a/include/boost/smart_ptr/make_local_shared_object.hpp b/include/boost/smart_ptr/make_local_shared_object.hpp index ab83d60..4545c5e 100644 --- a/include/boost/smart_ptr/make_local_shared_object.hpp +++ b/include/boost/smart_ptr/make_local_shared_object.hpp @@ -13,6 +13,7 @@ #include #include +#include #include #include #include @@ -186,12 +187,14 @@ template typename boost::detail::lsp_if_not_array::type all template typename boost::detail::lsp_if_not_array::type make_local_shared( Args&&... args ) { - return boost::allocate_local_shared( std::allocator(), std::forward(args)... ); + typedef typename boost::remove_const::type T2; + return boost::allocate_local_shared( std::allocator(), std::forward(args)... ); } template typename boost::detail::lsp_if_not_array::type make_local_shared_noinit() { - return boost::allocate_shared_noinit( std::allocator() ); + typedef typename boost::remove_const::type T2; + return boost::allocate_shared_noinit( std::allocator() ); } } // namespace boost diff --git a/test/Jamfile b/test/Jamfile index 0ede073..9e8fee6 100644 --- a/test/Jamfile +++ b/test/Jamfile @@ -283,3 +283,4 @@ run dll_test_main.cpp dll_test : : : static : dll_test_static ; run dll_test_main.cpp dll_test : : : shared : dll_test_shared ; run make_shared_const_test.cpp ; +run make_local_shared_const_test.cpp ; diff --git a/test/make_local_shared_const_test.cpp b/test/make_local_shared_const_test.cpp new file mode 100644 index 0000000..a57577e --- /dev/null +++ b/test/make_local_shared_const_test.cpp @@ -0,0 +1,262 @@ +// make_local_shared_const_test.cpp +// +// Copyright 2007-2009, 2017, 2018 Peter Dimov +// +// Distributed under the Boost Software License, Version 1.0. +// See accompanying file LICENSE_1_0.txt or copy at +// http://www.boost.org/LICENSE_1_0.txt + +#include + +#if defined( BOOST_NO_CXX11_RVALUE_REFERENCES ) || defined( BOOST_NO_CXX11_VARIADIC_TEMPLATES ) + +int main() +{ +} + +#else + +#include +#include +#include +#include +#include + +class X +{ +private: + + X( X const & ); + X & operator=( X const & ); + + void * operator new( std::size_t n ) + { + // lack of this definition causes link errors on Comeau C++ + BOOST_ERROR( "private X::new called" ); + return ::operator new( n ); + } + + void operator delete( void * p ) + { + // lack of this definition causes link errors on MSVC + BOOST_ERROR( "private X::delete called" ); + ::operator delete( p ); + } + +public: + + static int instances; + + int v; + + explicit X( int a1 = 0, int a2 = 0, int a3 = 0, int a4 = 0, int a5 = 0, int a6 = 0, int a7 = 0, int a8 = 0, int a9 = 0 ): v( a1+a2+a3+a4+a5+a6+a7+a8+a9 ) + { + ++instances; + } + + ~X() + { + --instances; + } +}; + +int X::instances = 0; + +template static long use_count( boost::local_shared_ptr const & p ) +{ + return boost::shared_ptr( p ).use_count() - 1; +} + +int main() +{ + { + boost::local_shared_ptr pi = boost::make_local_shared(); + + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + + BOOST_TEST( *pi == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 5 ); + + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + + BOOST_TEST( *pi == 5 ); + } + + BOOST_TEST( X::instances == 0 ); + + { + boost::local_shared_ptr pi = boost::make_local_shared(); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 0 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared_noinit(); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 0 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3, 4 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3+4 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3, 4, 5 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3+4+5 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3, 4, 5, 6 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3+4+5+6 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3, 4, 5, 6, 7 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3+4+5+6+7 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3, 4, 5, 6, 7, 8 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3+4+5+6+7+8 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + { + boost::local_shared_ptr pi = boost::make_local_shared( 1, 2, 3, 4, 5, 6, 7, 8, 9 ); + boost::weak_ptr wp( pi ); + + BOOST_TEST( X::instances == 1 ); + BOOST_TEST( pi.get() != 0 ); + BOOST_TEST( pi.local_use_count() == 1 ); + BOOST_TEST( use_count( pi ) == 1 ); + BOOST_TEST( pi->v == 1+2+3+4+5+6+7+8+9 ); + + pi.reset(); + + BOOST_TEST( X::instances == 0 ); + } + + return boost::report_errors(); +} + +#endif