[string_algo][patch] is_any_of fails to find ::memcpy in c++builder

[peter.myerscough-jackopson@…]

C++builder has a bug in its string.h and cstring headers such that if they are included in the 'wrong' order memcpy is no longer accessible in the global namespace, but is accessible in the std:: namespace. see this​http://mail-archives.apache.org/mod_mbox/xerces-c-users/200810.mbox/%3C48E5CA01.8050204@mebius.net%3E

Given that memcpy is included from cstring then std::memcpy is available and so this simple patch enables this to work in C++Builder.

change

128                    // Use fixed storage
129                    ::memcpy(DestStorage, SrcStorage, sizeof(set_value_type)*m_Size);
130                }

to

128                    // Use fixed storage
129                    ::std::memcpy(DestStorage, SrcStorage, sizeof(set_value_type)*m_Size);
130                }

AND

this

208                    // Copy the data
209                    ::memcpy(DestStorage, SrcStorage, sizeof(set_value_type)*m_Size);
210
211                    return *this;

to this

208                    // Copy the data
209                    ::std::memcpy(DestStorage, SrcStorage, sizeof(set_value_type)*m_Size);
210
211                    return *this;

This will enable the is_any_of to work again for C++Builder's broken headers. It will also improve the style of the header as all other calls to standard functions are called in the std namespace.

Thanks

Peter Myerscough-Jackopson

[Marshall Clow]

(In [76213]) Qualified two calls to memcpy to work around a C++Builder bug; Refs #4811

[Marshall Clow]

(In [76265]) Merge Change 76213 to release; Fixes #4811