Merge pull request #1536 from IntelPython/fix-copy-instead-of-move-coverity-issue

oleksandr-pavlyk · web-flow · commit f9d3501cd998 · 2024-02-07T16:41:58.000-06:00
Addressed COPY_INSTEAD_OF_MOVE coverity issue
diff --git a/dpctl/tensor/libtensor/include/kernels/sorting.hpp b/dpctl/tensor/libtensor/include/kernels/sorting.hpp
@@ -287,7 +287,8 @@ void leaf_sort_impl(Iter first,
                     const size_t end,
                     Compare comp)
 {
-    return insertion_sort_impl<Iter, Compare>(first, begin, end, comp);
+    return insertion_sort_impl<Iter, Compare>(
+        std::move(first), std::move(begin), std::move(end), std::move(comp));
 }
 } // namespace
 

Original file line number	Diff line number	Diff line change
`@@ -287,7 +287,8 @@ void leaf_sort_impl(Iter first,`
`287`	`287`	`const size_t end,`
`288`	`288`	`Compare comp)`
`289`	`289`	`{`
`290`		`- return insertion_sort_impl<Iter, Compare>(first, begin, end, comp);`
	`290`	`+ return insertion_sort_impl<Iter, Compare>(`
	`291`	`+ std::move(first), std::move(begin), std::move(end), std::move(comp));`
`291`	`292`	`}`
`292`	`293`	`} // namespace`
`293`	`294`