MT#59962 AmSessionContainer: move app_params

When building up the AmSession and assigning
the app parameters to it, do the move operation
instead of copying.

Fixes defect:

569         //  ... and do nothing !
570
571         DBG("onInvite/onRefer returned NULL\n");
572       }
573       else {
574         // save session parameters
>>>     CID 550234:  Performance inefficiencies  (COPY_INSTEAD_OF_MOVE)
>>>     "app_params" is copied in call to copy assignment for class
        "std::map<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >,
        std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >,
        std::less<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >,
        std::allocator<std::pair<std::__cxx11::basic_string<char, std::char_traits<char>,
        std::allocator<char> > const, std::__cxx11::basic_string<char, std::char_traits<char>,
        std::allocator<char> > > > >", when it could be moved instead.
575         session->app_params = app_params;
576       }
577
578       return session;
579     }
580

Change-Id: I3502a4c36ab6ef24fe53d57e2d5fb01777232b2b
mr13.4
Donat Zenichev 1 year ago
parent 357b2045d6
commit db1c069cc6

@ -572,7 +572,7 @@ AmSession* AmSessionContainer::createSession(const AmSipRequest& req,
}
else {
// save session parameters
session->app_params = app_params;
session->app_params = std::move(app_params);
}
return session;

Loading…
Cancel
Save