fbcode/fboss/agent/state

Reviewed By: jasmeetbagga

Differential Revision: D90976317

fbshipit-source-id: 3173d66d341de0e6584a897401f0a8f0923f2aeb

Author: generatedunixname537391475639613

fboss/agent/state/Route.h

@@ -322,6 +322,8 @@ class Route : public ThriftStructNode<Route<AddrT>, ThriftFieldsT<AddrT>> {
     nexthopsmulti->update(clientId, entry);
   }
 
+  ~Route() = default;
+
   template <typename... Args>
   static ThriftFieldsT<AddrT> makeThrift(Args&&... args) {
     auto fields = LegacyFields(std::forward<Args>(args)...);
@@ -530,6 +532,8 @@ class Route : public ThriftStructNode<Route<AddrT>, ThriftFieldsT<AddrT>> {
   // no copy or assign operator
   Route(const Route&) = delete;
   Route& operator=(const Route&) = delete;
+  Route(Route&&) = delete;
+  Route& operator=(Route&&) = delete;
 
   // Inherit the constructors required for clone()
   using RouteBase::RouteBase;

fboss/agent/state/StateUpdate.h

@@ -41,7 +41,7 @@ class StateUpdate {
       static_cast<int>(BehaviorFlags::NONE);
   explicit StateUpdate(folly::StringPiece name, int behaviorFlags)
       : name_(name.str()), behaviorFlags_(behaviorFlags) {}
-  virtual ~StateUpdate() {}
+  virtual ~StateUpdate() = default;
 
   const std::string& getName() const {
     return name_;
@@ -95,6 +95,8 @@ class StateUpdate {
   // Forbidden copy constructor and assignment operator
   StateUpdate(StateUpdate const&) = delete;
   StateUpdate& operator=(StateUpdate const&) = delete;
+  StateUpdate(StateUpdate&&) = delete;
+  StateUpdate& operator=(StateUpdate&&) = delete;
 
   std::string name_;
   int behaviorFlags_{static_cast<int>(BehaviorFlags::NONE)};