Skip to content

Commit

Permalink
Removing std::move use with const object [ci skip] (#848)
Browse files Browse the repository at this point in the history
* Removing std::move use with const [ci skip]

Signed-off-by: Tahar Touati <touati.thr@gmail.com>

* Addressing PR feedback, removing const from complex types and adding
std
move back [ci skip]

Signed-off-by: Tahar Touati <touati.thr@gmail.com>

---------

Signed-off-by: Tahar Touati <touati.thr@gmail.com>
  • Loading branch information
thrtouati committed May 8, 2023
1 parent 6df4810 commit 3f2a461
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,10 @@ namespace cppmicroservices
class ConfigurationEvent
{
public:
ConfigurationEvent(ServiceReference<ConfigurationAdmin> const configAdmin,
ConfigurationEvent(ServiceReference<ConfigurationAdmin> configAdmin,
const ConfigurationEventType type,
const std::string factoryPid,
const std::string pid)
std::string factoryPid,
std::string pid)
: configAdmin(std::move(configAdmin))
, type(type)
, factoryPid(std::move(factoryPid))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ namespace cppmicroservices
{
ListenerToken(std::string pid, const ListenerTokenId tokenId)
: pid(std::move(pid))
, tokenId(std::move(tokenId))
, tokenId(tokenId)
{
}
std::string pid;
Expand Down

0 comments on commit 3f2a461

Please sign in to comment.