Remove unnecessary copy of preAllocatedOutputTensor (#1026)

mollyxu · Molly Xu · web-flow · commit 7d27c15993d3 · 2025-11-06T16:50:49.000-05:00
Co-authored-by: Molly Xu &lt;mollyxu@fb.com&gt;
diff --git a/src/torchcodec/_core/SingleStreamDecoder.cpp b/src/torchcodec/_core/SingleStreamDecoder.cpp
@@ -1296,7 +1296,7 @@ FrameOutput SingleStreamDecoder::convertAVFrameToFrameOutput(
       getDuration(avFrame),
       formatContext_->streams[activeStreamIndex_]->time_base);
   deviceInterface_->convertAVFrameToFrameOutput(
-      avFrame, frameOutput, preAllocatedOutputTensor);
+      avFrame, frameOutput, std::move(preAllocatedOutputTensor));
   return frameOutput;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1296,7 +1296,7 @@ FrameOutput SingleStreamDecoder::convertAVFrameToFrameOutput(`
`1296`	`1296`	`getDuration(avFrame),`
`1297`	`1297`	`formatContext_->streams[activeStreamIndex_]->time_base);`
`1298`	`1298`	`deviceInterface_->convertAVFrameToFrameOutput(`
`1299`		`- avFrame, frameOutput, preAllocatedOutputTensor);`
	`1299`	`+ avFrame, frameOutput, std::move(preAllocatedOutputTensor));`
`1300`	`1300`	`return frameOutput;`
`1301`	`1301`	`}`
`1302`	`1302`