Boost logo

Boost-Commit :

Subject: [Boost-commit] svn:boost r80223 - trunk/boost/unordered/detail
From: dnljms_at_[hidden]
Date: 2012-08-25 17:53:54


Author: danieljames
Date: 2012-08-25 17:53:53 EDT (Sat, 25 Aug 2012)
New Revision: 80223
URL: http://svn.boost.org/trac/boost/changeset/80223

Log:
Unordered: Rename `*_impl` to `*_value_impl`.

To make it clear that they should only be used to construct and destroy
the value, and not nodes or buckets.
Text files modified:
   trunk/boost/unordered/detail/allocate.hpp | 32 ++++++++++++++++----------------
   trunk/boost/unordered/detail/buckets.hpp | 10 +++++-----
   2 files changed, 21 insertions(+), 21 deletions(-)

Modified: trunk/boost/unordered/detail/allocate.hpp
==============================================================================
--- trunk/boost/unordered/detail/allocate.hpp (original)
+++ trunk/boost/unordered/detail/allocate.hpp 2012-08-25 17:53:53 EDT (Sat, 25 Aug 2012)
@@ -776,7 +776,7 @@
     }
 
     template <typename Alloc, typename T>
- inline void destroy_impl(Alloc& alloc, T* x) {
+ inline void destroy_value_impl(Alloc& alloc, T* x) {
         boost::unordered::detail::allocator_traits<Alloc>::destroy(alloc, x);
     }
 
@@ -791,7 +791,7 @@
     }
 
     template <typename Alloc, typename T>
- inline void destroy_impl(Alloc&, T* x) {
+ inline void destroy_value_impl(Alloc&, T* x) {
         boost::unordered::detail::destroy(x);
     }
 
@@ -801,7 +801,7 @@
 #else
 
     template <typename Alloc, typename T>
- inline void destroy_impl(Alloc&, T* x) {
+ inline void destroy_value_impl(Alloc&, T* x) {
         boost::unordered::detail::destroy(x);
     }
 
@@ -978,7 +978,7 @@
     // Construct from variadic parameters
 
     template <typename Alloc, typename T, typename... Args>
- inline void construct_impl(Alloc& alloc, T* address,
+ inline void construct_value_impl(Alloc& alloc, T* address,
         BOOST_FWD_REF(Args)... args)
     {
         boost::unordered::detail::call_construct(alloc,
@@ -988,7 +988,7 @@
     template <typename Alloc, typename A, typename B,
         typename A0, typename A1, typename A2>
     inline typename enable_if<piecewise3<A, B, A0>, void>::type
- construct_impl(Alloc& alloc, std::pair<A, B>* address,
+ construct_value_impl(Alloc& alloc, std::pair<A, B>* address,
             BOOST_FWD_REF(A0), BOOST_FWD_REF(A1) a1, BOOST_FWD_REF(A2) a2)
     {
         boost::unordered::detail::construct_from_tuple(alloc,
@@ -1001,7 +1001,7 @@
 
     template <typename Alloc, typename A, typename B, typename A0>
     inline typename enable_if<emulation1<A, B, A0>, void>::type
- construct_impl(Alloc& alloc, std::pair<A, B>* address,
+ construct_value_impl(Alloc& alloc, std::pair<A, B>* address,
             BOOST_FWD_REF(A0) a0)
     {
         boost::unordered::detail::call_construct(alloc,
@@ -1013,7 +1013,7 @@
     template <typename Alloc, typename A, typename B,
         typename A0, typename A1, typename A2>
     inline typename enable_if<emulation3<A, B, A0>, void>::type
- construct_impl(Alloc& alloc, std::pair<A, B>* address,
+ construct_value_impl(Alloc& alloc, std::pair<A, B>* address,
             BOOST_FWD_REF(A0) a0, BOOST_FWD_REF(A1) a1, BOOST_FWD_REF(A2) a2)
     {
         boost::unordered::detail::call_construct(alloc,
@@ -1027,7 +1027,7 @@
     template <typename Alloc, typename A, typename B,
             typename A0, typename A1, typename A2, typename A3,
             typename... Args>
- inline void construct_impl(Alloc& alloc, std::pair<A, B>* address,
+ inline void construct_value_impl(Alloc& alloc, std::pair<A, B>* address,
             BOOST_FWD_REF(A0) a0, BOOST_FWD_REF(A1) a1, BOOST_FWD_REF(A2) a2,
             BOOST_FWD_REF(A3) a3, BOOST_FWD_REF(Args)... args)
     {
@@ -1052,7 +1052,7 @@
         typename Alloc, typename T, \
         BOOST_PP_ENUM_PARAMS_Z(z, num_params, typename A) \
> \
- inline void construct_impl(Alloc&, T* address, \
+ inline void construct_value_impl(Alloc&, T* address, \
         boost::unordered::detail::BOOST_PP_CAT(emplace_args,num_params) < \
             BOOST_PP_ENUM_PARAMS_Z(z, num_params, A) \
> const& args) \
@@ -1063,14 +1063,14 @@
     }
 
     template <typename Alloc, typename T, typename A0>
- inline void construct_impl(Alloc&, T* address,
+ inline void construct_value_impl(Alloc&, T* address,
             emplace_args1<A0> const& args)
     {
         new((void*) address) T(boost::forward<A0>(args.a0));
     }
 
     template <typename Alloc, typename T, typename A0, typename A1>
- inline void construct_impl(Alloc&, T* address,
+ inline void construct_value_impl(Alloc&, T* address,
             emplace_args2<A0, A1> const& args)
     {
         new((void*) address) T(
@@ -1080,7 +1080,7 @@
     }
 
     template <typename Alloc, typename T, typename A0, typename A1, typename A2>
- inline void construct_impl(Alloc&, T* address,
+ inline void construct_value_impl(Alloc&, T* address,
             emplace_args3<A0, A1, A2> const& args)
     {
         new((void*) address) T(
@@ -1097,7 +1097,7 @@
 
     template <typename Alloc, typename A, typename B,
         typename A0, typename A1, typename A2>
- inline void construct_impl(Alloc& alloc, std::pair<A, B>* address,
+ inline void construct_value_impl(Alloc& alloc, std::pair<A, B>* address,
             boost::unordered::detail::emplace_args3<A0, A1, A2> const& args,
             typename enable_if<piecewise3<A, B, A0>, void*>::type = 0)
     {
@@ -1110,7 +1110,7 @@
 #if defined(BOOST_UNORDERED_DEPRECATED_PAIR_CONSTRUCT)
 
     template <typename Alloc, typename A, typename B, typename A0>
- inline void construct_impl(Alloc&, std::pair<A, B>* address,
+ inline void construct_value_impl(Alloc&, std::pair<A, B>* address,
             boost::unordered::detail::emplace_args1<A0> const& args,
             typename enable_if<emulation1<A, B, A0>, void*>::type = 0)
     {
@@ -1121,7 +1121,7 @@
 
     template <typename Alloc, typename A, typename B,
         typename A0, typename A1, typename A2>
- inline void construct_impl(Alloc&, std::pair<A, B>* address,
+ inline void construct_value_impl(Alloc&, std::pair<A, B>* address,
             boost::unordered::detail::emplace_args3<A0, A1, A2> const& args,
             typename enable_if<emulation3<A, B, A0>, void*>::type = 0)
     {
@@ -1136,7 +1136,7 @@
     template <typename Alloc, typename A, typename B, \
         BOOST_PP_ENUM_PARAMS_Z(z, num_params, typename A) \
> \
- inline void construct_impl(Alloc&, std::pair<A, B>* address, \
+ inline void construct_value_impl(Alloc&, std::pair<A, B>* address, \
         boost::unordered::detail::BOOST_PP_CAT(emplace_args, num_params) < \
                 BOOST_PP_ENUM_PARAMS_Z(z, num_params, A) \
> const& args) \

Modified: trunk/boost/unordered/detail/buckets.hpp
==============================================================================
--- trunk/boost/unordered/detail/buckets.hpp (original)
+++ trunk/boost/unordered/detail/buckets.hpp 2012-08-25 17:53:53 EDT (Sat, 25 Aug 2012)
@@ -74,7 +74,7 @@
         void construct_value(BOOST_UNORDERED_EMPLACE_ARGS)
         {
             BOOST_ASSERT(node_ && node_constructed_ && !value_constructed_);
- boost::unordered::detail::construct_impl(
+ boost::unordered::detail::construct_value_impl(
                 alloc_, node_->value_ptr(), BOOST_UNORDERED_EMPLACE_FORWARD);
             value_constructed_ = true;
         }
@@ -83,7 +83,7 @@
         void construct_value2(BOOST_FWD_REF(A0) a0)
         {
             BOOST_ASSERT(node_ && node_constructed_ && !value_constructed_);
- boost::unordered::detail::construct_impl(
+ boost::unordered::detail::construct_value_impl(
                 alloc_, node_->value_ptr(),
                 BOOST_UNORDERED_EMPLACE_ARGS1(boost::forward<A0>(a0)));
             value_constructed_ = true;
@@ -112,7 +112,7 @@
     {
         if (node_) {
             if (value_constructed_) {
- boost::unordered::detail::destroy_impl(alloc_,
+ boost::unordered::detail::destroy_value_impl(alloc_,
                     node_->value_ptr());
             }
 
@@ -144,7 +144,7 @@
 
             if (value_constructed_)
             {
- boost::unordered::detail::destroy_impl(alloc_,
+ boost::unordered::detail::destroy_value_impl(alloc_,
                     node_->value_ptr());
                 value_constructed_ = false;
             }
@@ -764,7 +764,7 @@
 
         inline void delete_node(c_iterator n)
         {
- boost::unordered::detail::destroy_impl(node_alloc(),
+ boost::unordered::detail::destroy_value_impl(node_alloc(),
                 n.node_->value_ptr());
             node_allocator_traits::destroy(node_alloc(),
                     boost::addressof(*n.node_));


Boost-Commit list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk