#962 send- cleanup notes and such

src/vt/messaging/active.h

@@ -185,12 +185,17 @@ struct BufferedActiveMsg {
  */
 template <typename MsgT>
 struct TransferMsg {
-  // TODO: This should 'go away' when direct 'msg.get()' usages are replaced with 'msg'
+  // This should go away entirely once all usages are eliminated.
+  // The core usages are for tests to ensure the current API works,
   // [[deprecated]]
   /*implicit*/ TransferMsg(MsgT const* msgPtr) : msg_(promoteMsg<MsgT>(const_cast<MsgT*>(msgPtr))) {
   }
 
-  // n.b. invalidates msg
+  /**
+   * \brief Moves msg into the transfer message.
+   *
+   * As though std::move(msg) was used, msg is invalidated.
+   */
   /*implicit*/ TransferMsg(MsgPtr<MsgT>& msg) : msg_(std::move(msg)) {
   }