std::move in ctor initializer

move.cpp

#include <utility>
#include <iostream>

class ctor
{
public:
    ctor()
    {
        std::cout << "default ctor" << std::endl;
    }

    ctor(const ctor &a)
    {
        std::cout << "copy ctor" << std::endl;
    }

    ctor(ctor &&a)
    {
        std::cout << "move ctor" << std::endl;
    }

};

class ctorholder
{
public:
    ctorholder(ctor &&a) :
        mctor(a)
    {
        // param a is an lvalue here: a named variable is an lvalue.
        std::cout << "copy init" << std::endl;
    }

    ctorholder(ctor &a) :
        mctor(std::move(a))
    {
        // This would work
        std::cout << "move init" << std::endl;
    }


    ctor mctor;
};

int main()
{
    ctor a;

    ctorholder holder1(a);                // Correct
    ctorholder holder2(std::move(a));     // Doesn't work. calls copy ctor

    return 0;

}

See https://stackoverflow.com/questions/13138851/why-do-i-need-to-use-stdmove-in-the-initialization-list-of-a-move-constructor

From std::vector.
We did it correctly.