*xpos = mystl::move(value_copy);建议修改

[shuiyihang]

vector插入函数：vector::insert(const_iterator pos, const value_type& value)
else if (end_ != cap_)域中使用了*xpos = mystl::move(value_copy);

我觉得这里不太妥当，首先代码结果没有问题，但是会给阅读者造成不必要困惑。

举个例子有一个自定义类型，实现了移动赋值运算
`auto operator=(test_1&& other) noexcept ->test_1&
{
std::cout << "move\n";
b = other.b;
c = other.c;
other.b = ' ';
other.c = ' ';

    return *this;
}`

我们来分析下：

auto value_copy = value;
xpos = mystl::move(value_copy);
这个过程中先调用了拷贝构造，又使用了移动赋值运算。注释是担心value被改变，所以临时生成一个copy，是的他很有作用。
但为什么不直接xpos = mystl::move(value);呢，因为这个insert插入的值是const的，所以最终调用的还是copy构造等价于
*xpos = value；

我认为现在的代码中具有迷惑性，不如去掉move，直接*xpos = value；

[Peas-Li]

因为value可能会被改变，这里的value可能是引用的该vector对象中pos位置后的值，copy_backward后，value的值也就变了，所以必须提前拷贝