[FOLD] Address @seelabs feedback

* Change STObject::set() arg to rvalue ref

Author: gregtatcam

src/ripple/app/misc/impl/AMMUtils.cpp

@@ -314,7 +314,7 @@ initializeFeeAuctionVote(
             {
                 STObject auctionSlot =
                     STObject::makeInnerObject(sfAuctionSlot, rules);
-                ammSle->set(&auctionSlot);
+                ammSle->set(std::move(auctionSlot));
             }
             return ammSle->peekFieldObject(sfAuctionSlot);
         }

src/ripple/protocol/STObject.h

@@ -343,7 +343,7 @@ class STObject : public STBase, public CountedObject<STObject>
     set(std::unique_ptr<STBase> v);
 
     void
-    set(STBase* v);
+    set(STBase&& v);
 
     void
     setFieldU8(SField const& field, unsigned char);

src/ripple/protocol/impl/STObject.cpp

@@ -645,23 +645,22 @@ STObject::getFieldArray(SField const& field) const
 void
 STObject::set(std::unique_ptr<STBase> v)
 {
-    set(v.get());
+    set(std::move(*v.get()));
 }
 
 void
-STObject::set(STBase* v)
+STObject::set(STBase&& v)
 {
-    assert(v);
-    auto const i = getFieldIndex(v->getFName());
+    auto const i = getFieldIndex(v.getFName());
     if (i != -1)
     {
-        v_[i] = std::move(*v);
+        v_[i] = std::move(v);
     }
     else
     {
         if (!isFree())
             Throw<std::runtime_error>("missing field in templated STObject");
-        v_.emplace_back(std::move(*v));
+        v_.emplace_back(std::move(v));
     }
 }