From 316b4921a916e0e5496f45a77c41548b5ad0bb56 Mon Sep 17 00:00:00 2001 From: Andreas Buhr Date: Tue, 3 Feb 2026 11:25:50 +0100 Subject: [PATCH] Move values into detail::assign Code cleanup: Most of the time, values are std::moved into detail::assign, but not always. This patch makes usage more consistent to always use std::move, except for ints, floats, and iterators. --- include/boost/parser/parser.hpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/boost/parser/parser.hpp b/include/boost/parser/parser.hpp index bfa1af76..2628c93a 100644 --- a/include/boost/parser/parser.hpp +++ b/include/boost/parser/parser.hpp @@ -3716,7 +3716,7 @@ namespace boost { namespace parser { if constexpr (detail::is_struct_compatible_v< Attribute, result_t>) { - detail::assign(retval, temp_retval); + detail::assign(retval, std::move(temp_retval)); } else { detail::assign( retval, @@ -5488,7 +5488,7 @@ namespace boost { namespace parser { dont_assign); if (success && !dont_assign) { if constexpr (!detail::is_nope_v) - detail::assign(retval, attr); + detail::assign(retval, std::move(attr)); } } @@ -5583,7 +5583,7 @@ namespace boost { namespace parser { container && container) { detail::move_back(retval, attr, detail::gen_attrs(flags)); } else { - detail::assign(retval, attr); + detail::assign(retval, std::move(attr)); } } }