Merge branch 'nomove_const' into 'master'

Don't use std::move on const See merge request OpenMW/openmw!805
4 years ago · 9cc7b1f4bd
parent aca5d2fc12 d4d111a709
commit 9cc7b1f4bd
1 changed files with 2 additions and 2 deletions
--- a/components/detournavigator/asyncnavmeshupdater.cpp
+++ b/components/detournavigator/asyncnavmeshupdater.cpp
@ -262,7 +262,7 @@ namespace DetourNavigator
        if (jobs.top().mProcessTime > now)
            return {};

-        Job job = std::move(jobs.top());
+        Job job = jobs.top();
        jobs.pop();

        if (changeLastUpdate && job.mChangeType == ChangeType::update)
@ -273,7 +273,7 @@ namespace DetourNavigator
        if (it->second.empty())
            pushed.erase(it);

-        return {std::move(job)};
+        return job;
    }

    void AsyncNavMeshUpdater::writeDebugFiles(const Job& job, const RecastMesh* recastMesh) const