Mapper::findBy成员函数的优化建议

[windflowerly]

原drogon::orm::Mapper::findBy函数返回语句是这样写的

    .....
    std::vector<T> ret;
    for (auto const &row : r)
    {
        ret.push_back(T(row));
    }
    return ret;

此处是一个值传递，会引起重复的拷贝动作。可否换成移动语义，减少拷贝

[an-tao]

没太明白，您是指用emplace_back代替push_back?

[windflowerly]

应该是我没有描述清楚。
这个函数的返回值是个值类型。是不是应该 return std::move(ret);

如果查询返回的结果集较大，则会引起大量的拷贝。

当然，这里也应该用emplce_back.

[an-tao]

不会的，编译器会优化，您可以搜索下RVO，谢谢反馈

[kaili302]

同意@an-tao, 但好像还是可以稍微优化下ret.reserve(r.size())？

[kaili302]

@an-tao @windflowerly please review: #2510