Remove redundant std::move() (dotnet/core-setup#5824)
authorOmair Majid <omajid@redhat.com>
Sat, 13 Apr 2019 09:45:08 +0000 (05:45 -0400)
committerVitek Karas <vitek.karas@microsoft.com>
Sat, 13 Apr 2019 09:45:08 +0000 (02:45 -0700)
commit03a37d7c406ec5e174e0a2ec727e5512612a26bd
treea851f370d2595f874aa76e97907c4c55812c2a8b
parentdda9df5f585f8725757e4753d1666824fe07141c
Remove redundant std::move() (dotnet/core-setup#5824)

This is essentially a port of:
https://github.com/Microsoft/cpprestsdk/commit/dotnet/core-setup@f0637654fc0e6752d60f483138cbc8aef4ff0b75#diff-8e55486b33944293c131ee21fe87ff37

clang (and gcc, but I didn't test that) complains about std::move being
redundant because the compiler can already infer it.

Commit migrated from https://github.com/dotnet/core-setup/commit/ded9cda79f8c48f60c56596f8badd99feffd98a3
src/installer/corehost/cli/json/casablanca/src/utilities/asyncrt_utils.cpp