chore: remove std::move for a temporary object

SYaoJun · SYaoJun · commit ab2938a78d5d · 2026-02-12T16:57:38.000+08:00
diff --git a/src/iceberg/expression/manifest_evaluator.cc b/src/iceberg/expression/manifest_evaluator.cc
@@ -340,7 +340,7 @@ class ManifestEvalVisitor : public BoundVisitor<bool> {
       return NotSupported("Bounds of non-primitive partition fields are not supported.");
     }
     return Literal::Deserialize(
-        bound, std::move(internal::checked_pointer_cast<PrimitiveType>(type)));
+        bound, internal::checked_pointer_cast<PrimitiveType>(type)));
   }
 
  private:

Original file line number	Diff line number	Diff line change
`@@ -340,7 +340,7 @@ class ManifestEvalVisitor : public BoundVisitor<bool> {`
`340`	`340`	`return NotSupported("Bounds of non-primitive partition fields are not supported.");`
`341`	`341`	`}`
`342`	`342`	`return Literal::Deserialize(`
`343`		`- bound, std::move(internal::checked_pointer_cast<PrimitiveType>(type)));`
	`343`	`+ bound, internal::checked_pointer_cast<PrimitiveType>(type)));`
`344`	`344`	`}`
`345`	`345`
`346`	`346`	`private:`