[libcxx] r185863 - Implement n3668 - std::exchange
Richard Smith
richard at metafoo.co.uk
Mon Jul 8 14:12:00 PDT 2013
On Mon, Jul 8, 2013 at 1:54 PM, Marshall Clow <mclow.lists at gmail.com> wrote:
> Author: marshall
> Date: Mon Jul 8 15:54:40 2013
> New Revision: 185863
>
> URL: http://llvm.org/viewvc/llvm-project?rev=185863&view=rev
> Log:
> Implement n3668 - std::exchange
>
> Added:
> libcxx/trunk/test/utilities/utility/exchange/
> libcxx/trunk/test/utilities/utility/exchange/exchange.pass.cpp
> Modified:
> libcxx/trunk/include/utility
>
> Modified: libcxx/trunk/include/utility
> URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/utility?rev=185863&r1=185862&r2=185863&view=diff
> ==============================================================================
> --- libcxx/trunk/include/utility (original)
> +++ libcxx/trunk/include/utility Mon Jul 8 15:54:40 2013
> @@ -138,6 +138,8 @@ template<size_t N>
> template<class... T>
> using index_sequence_for = make_index_sequence<sizeof...(T)>;
>
> +template<class T, class U=T>
> + T exchange(T& obj, U&& new_value);
> } // std
>
> */
> @@ -685,6 +687,17 @@ template<class... _Tp>
>
> #endif // _LIBCPP_STD_VER > 11
>
> +#if _LIBCPP_STD_VER > 11
> +template<class _T1, class _T2 = _T1>
> +_LIBCPP_INLINE_VISIBILITY inline
> +_T1 exchange(_T1& __obj, _T2 && __new_value)
> +{
> + _T1 __old_value = _VSTD::move(__obj);
> + __obj = _VSTD::forward<_T2>(__new_value);
> + return __old_value;
> +}
> +#endif // _LIBCPP_STD_VER > 11
> +
> _LIBCPP_END_NAMESPACE_STD
>
> #endif // _LIBCPP_UTILITY
>
> Added: libcxx/trunk/test/utilities/utility/exchange/exchange.pass.cpp
> URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/utilities/utility/exchange/exchange.pass.cpp?rev=185863&view=auto
> ==============================================================================
> --- libcxx/trunk/test/utilities/utility/exchange/exchange.pass.cpp (added)
> +++ libcxx/trunk/test/utilities/utility/exchange/exchange.pass.cpp Mon Jul 8 15:54:40 2013
> @@ -0,0 +1,47 @@
> +//===----------------------------------------------------------------------===//
> +//
> +// The LLVM Compiler Infrastructure
> +//
> +// This file is dual licensed under the MIT and the University of Illinois Open
> +// Source Licenses. See LICENSE.TXT for details.
> +//
> +//===----------------------------------------------------------------------===//
> +
> +// utilities
> +
> +// exchange
> +
> +#include <utility>
> +#include <cassert>
> +#include <string>
> +
> +int main()
> +{
> +#if _LIBCPP_STD_VER > 11
> + {
> + int v = 12;
> + assert ( std::exchange ( v, 23 ) == 12 );
> + assert ( v == 23 );
> + assert ( std::exchange ( v, 67.2 ) == 23 );
> + assert ( v = 67 );
> + }
> +
> + {
> + bool b = false;
> + assert ( !std::exchange ( b, true ));
> + assert ( b );
> + }
> +
> + {
> + const std::string s1 ( "Hi Mom!" );
> + const std::string s2 ( "Yo Dad!" );
> + std::string s3 = s1; // Mom
> + assert ( std::exchange ( s3, s2 ) == s1 );
> + assert ( s3 == s2 );
> + assert ( std::exchange ( s3, "Hi Mom!" ) == s2 );
> + assert ( s3 == s1 );
> + assert ( std::exchange ( s3, "" ) == s1 );
> + assert ( s3.size () == 0 );
> + }
> +#endif
> +}
It'd be useful to also test std::exchange's default template argument
trick, for the case where U is not deducible. Something like:
std::exchange( s3, {} )
More information about the cfe-commits
mailing list