From 2e332012fb998816bd6e10d15f8b84e620ec4b51 Mon Sep 17 00:00:00 2001
From: Chris Bracken <chris@bracken.jp>
Date: Fri, 17 Jun 2022 21:37:36 -0700
Subject: [PATCH] Eliminate use-after-move

pipeline_desc is a std::optional<PipelineDescriptor> looks like it can
be passed by value rather than moved.
---
 impeller/renderer/renderer_unittests.cc | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/impeller/renderer/renderer_unittests.cc b/impeller/renderer/renderer_unittests.cc
index 8f350d10bd1f0..539637d047f91 100644
--- a/impeller/renderer/renderer_unittests.cc
+++ b/impeller/renderer/renderer_unittests.cc
@@ -285,9 +285,8 @@ TEST_P(RendererTest, CanRenderToTexture) {
   auto pipeline_desc =
       BoxPipelineBuilder::MakeDefaultPipelineDescriptor(*context);
   ASSERT_TRUE(pipeline_desc.has_value());
-  auto box_pipeline = context->GetPipelineLibrary()
-                          ->GetRenderPipeline(std::move(pipeline_desc))
-                          .get();
+  auto box_pipeline =
+      context->GetPipelineLibrary()->GetRenderPipeline(pipeline_desc).get();
   ASSERT_TRUE(box_pipeline);
 
   VertexBufferBuilder<VS::PerVertexData> vertex_builder;