Opened 8 years ago
Closed 8 years ago
#11075 closed Bugs (fixed)
Unqualified calls to memmove/memcpy in container/detail/copy_move_algo.hpp
Reported by: | Owned by: | Ion Gaztañaga | |
---|---|---|---|
Milestone: | To Be Determined | Component: | container |
Version: | Boost Development Trunk | Severity: | Problem |
Keywords: | Cc: |
Description
Compiling libs/thread/test/sync/mutual_exclusion/sync_queue/single_thread_pass.cpp with Oracle Solaris Studio12.4 on Solaris 11.2, we see the following error:
"../boost/container/detail/copy_move_algo.hpp", line 177: Error: memmove is not defined. "../boost/container/detail/copy_move_algo.hpp", line 304: Where: While instantiating "boost::container::container_detail::memmove<int*, int*>(int*, int*, int*)". "../boost/container/detail/copy_move_algo.hpp", line 304: Where: Instantiated from boost::container::vector<int, boost::container::new_allocator<int>>::priv_forward_range_insert_new_allocation<boost::container::container_detail::insert_copy_proxy<boost::container::new_allocator<int>, int*>>(int*const, unsigned, int*const, const unsigned, boost::container::container_detail::insert_copy_proxy<boost::container::new_allocator<int>, int*>). "../boost/container/vector.hpp", line 2456: Where: Instantiated from non-template code.
In boost/container/detail/copy_move_algo.hpp, the header <cstring> is included, however calls to memmove and memcpy need to be qualified with std::
fixed in https://github.com/boostorg/container/commit/c194a6794db1f5428465eb12bf7e5220a0630bfa
Thanks for the patch.