SERVER-32603 Modernize QueryPlanner methods to use unique_ptr.

1 files changed, 5 insertions, 5 deletions

diff --git a/src/mongo/dbtests/plan_ranking.cpp b/src/mongo/dbtests/plan_ranking.cpp
index 80399f73b06..affc2bca9d0 100644
--- a/src/mongo/dbtests/plan_ranking.cpp
+++ b/src/mongo/dbtests/plan_ranking.cpp
@@ -115,9 +115,9 @@ public:
         plannerParams.options &= ~QueryPlannerParams::KEEP_MUTATIONS;
 
         // Plan.
-        vector<QuerySolution*> solutions;
-        Status status = QueryPlanner::plan(*cq, plannerParams, &solutions);
-        ASSERT(status.isOK());
+        auto statusWithSolutions = QueryPlanner::plan(*cq, plannerParams);
+        ASSERT_OK(statusWithSolutions.getStatus());
+        auto solutions = std::move(statusWithSolutions.getValue());
 
         ASSERT_GREATER_THAN_OR_EQUALS(solutions.size(), 1U);
 
@@ -128,8 +128,8 @@ public:
         for (size_t i = 0; i < solutions.size(); ++i) {
             PlanStage* root;
             ASSERT(StageBuilder::build(&_opCtx, collection, *cq, *solutions[i], ws.get(), &root));
-            // Takes ownership of all (actually some) arguments.
-            _mps->addPlan(solutions[i], root, ws.get());
+            // Takes ownership of 'root'.
+            _mps->addPlan(std::move(solutions[i]), root, ws.get());
         }
         // This is what sets a backup plan, should we test for it.
         PlanYieldPolicy yieldPolicy(PlanExecutor::NO_YIELD,