From 41600721629e3696369f5e524ea686d64c8f2b7c Mon Sep 17 00:00:00 2001 From: Donat Zenichev Date: Fri, 25 Apr 2025 14:22:09 +0200 Subject: [PATCH] MT#59962 ModConference: use move instead of copy For locally allocated AmArg objects, instead of copying just move them into the DSM session vars. Change-Id: Id75ac5f473b0dfb67ebb5b21189793288bfbc707 --- apps/dsm/mods/mod_conference/ModConference.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/apps/dsm/mods/mod_conference/ModConference.cpp b/apps/dsm/mods/mod_conference/ModConference.cpp index 6c0bb660..b2c09b88 100644 --- a/apps/dsm/mods/mod_conference/ModConference.cpp +++ b/apps/dsm/mods/mod_conference/ModConference.cpp @@ -170,7 +170,7 @@ EXEC_ACTION_START(ConfJoinAction) { // save channel for later use AmArg c_arg; c_arg.setBorrowedPointer(dsm_chan); - sc_sess->avar[CONF_AKEY_CHANNEL] = c_arg; + sc_sess->avar[CONF_AKEY_CHANNEL] = std::move(c_arg); // add to garbage collector sc_sess->transferOwnership(dsm_chan); @@ -279,7 +279,7 @@ EXEC_ACTION_START(ConfTeeJoinAction) { // remember DSMTeeConfChannel in session avar AmArg c_arg; c_arg.setBorrowedPointer(chan); - sc_sess->avar[conf_varname] = c_arg; + sc_sess->avar[conf_varname] = std::move(c_arg); // add to garbage collector sc_sess->transferOwnership(chan); @@ -380,7 +380,7 @@ EXEC_ACTION_START(ConfSetupMixInAction) { m_obj = new DSMDisposableT(m); AmArg c_arg; c_arg.setBorrowedPointer(m_obj); - sc_sess->avar[CONF_AKEY_MIXER] = c_arg; + sc_sess->avar[CONF_AKEY_MIXER] = std::move(c_arg); // add to garbage collector sc_sess->transferOwnership(m_obj); @@ -429,7 +429,7 @@ EXEC_ACTION_START(ConfPlayMixInListAction) { l_obj = new DSMDisposableT(pl); AmArg c_arg; c_arg.setBorrowedPointer(l_obj); - sc_sess->avar[CONF_AKEY_MIXLIST] = c_arg; + sc_sess->avar[CONF_AKEY_MIXLIST] = std::move(c_arg); // add to garbage collector sc_sess->transferOwnership(l_obj);