From 9aa3f72128a0a570177c8291479123f82600c6e8 Mon Sep 17 00:00:00 2001
From: Jerry Gamache <jerry.gamache@autodesk.com>
Date: Tue, 6 Dec 2022 12:57:29 -0500
Subject: [PATCH] Fix unix builds

Unnecessary use of std::move.
---
 lib/mayaUsd/ufe/UsdUndoConnectionCommands.cpp | 20 ++++++++-----------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/lib/mayaUsd/ufe/UsdUndoConnectionCommands.cpp b/lib/mayaUsd/ufe/UsdUndoConnectionCommands.cpp
index 96253cfdf1..092ebff4a6 100644
--- a/lib/mayaUsd/ufe/UsdUndoConnectionCommands.cpp
+++ b/lib/mayaUsd/ufe/UsdUndoConnectionCommands.cpp
@@ -139,8 +139,8 @@ UsdUndoCreateConnectionCommand::UsdUndoCreateConnectionCommand(
     const Ufe::Attribute::Ptr& srcAttr,
     const Ufe::Attribute::Ptr& dstAttr)
     : Ufe::ConnectionResultUndoableCommand()
-    , _srcInfo(std::move(std::make_unique<Ufe::AttributeInfo>(srcAttr))
-    , _dstInfo(std::move(std::make_unique<Ufe::AttributeInfo>(dstAttr))
+    , _srcInfo(std::make_unique<Ufe::AttributeInfo>(srcAttr))
+    , _dstInfo(std::make_unique<Ufe::AttributeInfo>(dstAttr))
 {
     // Validation goes here when we find out the right set of business rules. Failure should result
     // in a exception being thrown.
@@ -261,18 +261,16 @@ Ufe::Connection::Ptr UsdUndoCreateConnectionCommand::connection() const
     }
 }
 
-void UsdUndoCreateConnectionCommand::undo() {
-    _undoableItem.undo(); }
+void UsdUndoCreateConnectionCommand::undo() { _undoableItem.undo(); }
 
-void UsdUndoCreateConnectionCommand::redo() {
-    _undoableItem.redo(); }
+void UsdUndoCreateConnectionCommand::redo() { _undoableItem.redo(); }
 
 UsdUndoDeleteConnectionCommand::UsdUndoDeleteConnectionCommand(
     const Ufe::Attribute::Ptr& srcAttr,
     const Ufe::Attribute::Ptr& dstAttr)
     : Ufe::UndoableCommand()
-    , _srcInfo(std::move(std::make_unique<Ufe::AttributeInfo>(srcAttr))
-    , _dstInfo(std::move(std::make_unique<Ufe::AttributeInfo>(dstAttr))
+    , _srcInfo(std::make_unique<Ufe::AttributeInfo>(srcAttr))
+    , _dstInfo(std::make_unique<Ufe::AttributeInfo>(dstAttr))
 {
     // Validation goes here when we find out the right set of business rules. Failure should result
     // in a exception being thrown.
@@ -336,11 +334,9 @@ void UsdUndoDeleteConnectionCommand::execute()
     }
 }
 
-void UsdUndoDeleteConnectionCommand::undo() {
-    _undoableItem.undo(); }
+void UsdUndoDeleteConnectionCommand::undo() { _undoableItem.undo(); }
 
-void UsdUndoDeleteConnectionCommand::redo() {
-    _undoableItem.redo(); }
+void UsdUndoDeleteConnectionCommand::redo() { _undoableItem.redo(); }
 
 } // namespace ufe
 } // namespace MAYAUSD_NS_DEF