Skip to content

Commit

Permalink
remove std::move()
Browse files Browse the repository at this point in the history
Summary:
follow up from https://www.internalfb.com/diff/D67976583?dst_version_fbid=541757102157105&transaction_fbid=1663800114483059

simonmar:
> oh I just noticed, this std::move() is redundant (it's an integral type so no need to move it)

Reviewed By: bochko

Differential Revision: D68094334

fbshipit-source-id: 3b7bec7cec0568ce980b67af2f7c2f4674b43e18
  • Loading branch information
Kinga Bojarczuk authored and facebook-github-bot committed Jan 13, 2025
1 parent 7c25c8d commit 8c1ae4e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion glean/rts/query.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -495,7 +495,7 @@ std::unique_ptr<QueryResults> QueryExecutor::finish(
res->nested_fact_pids = std::move(nested_result_pids);
res->nested_fact_keys = std::move(nested_result_keys);
res->nested_fact_values = std::move(nested_result_values);
res->result_bytes = std::move(result_bytes);
res->result_bytes = result_bytes;

if (cont.hasValue()) {
res->continuation = cont->string();
Expand Down

0 comments on commit 8c1ae4e

Please sign in to comment.