Advantages of pass-by-value and std::move over pass-by-reference https://stackoverflow.com/questions/51705967/advantages-of-pass-by-value-and-stdmove-over-pass-by-reference

main.cpp

struct XYZ {
  XYZ(std::unique_ptr<std::string> str) : m_str (std::move(str)){}
  std::unique_ptr<std::string> m_str;
};

XYZ xyz(std::make_unique<std::string>("123"));
BOOST_CHECK_EQUAL(*(xyz.m_str), "fggfgf");

https://stackoverflow.com/a/22334896

f you are looking for the same functionality, the two variants should look as follows:

struct X
{
    Y data_;
    explicit X(const Y& data) : data_(data) { }
    explicit X(Y&& data) : data_(std::move(data)) { }
};

struct X
{
    Y data_;
    explicit X(Y data) : data_(std::move(data)) { }
};

The first variant saves one move operation, whereas the second variant is less to write. So, the answer is: Use the latter as long as you have no reason to optimize the performance.