Remove trailing semicolon in empty function bodies

src/core/model/pair.h

@@ -63,7 +63,9 @@ class PairValue : public AttributeValue
     typedef typename std::pair<first_type, second_type> result_type;
 
     PairValue()
-        : m_value(std::make_pair(Create<A>(), Create<B>())){};
+        : m_value(std::make_pair(Create<A>(), Create<B>()))
+    {
+    }
 
     /**
      * Construct this PairValue from a std::pair

src/core/model/simulator-impl.h

@@ -95,7 +95,9 @@ class SimulatorImpl : public Object
      *
      * \param [in] id The event about to be processed.
      */
-    virtual void PreEventHook(const EventId& id){};
+    virtual void PreEventHook(const EventId& id)
+    {
+    }
 };
 
 } // namespace ns3

src/core/test/simulator-test-suite.cc

@@ -211,13 +211,20 @@ class SimulatorTemplateTestCase : public TestCase
 {
   public:
     SimulatorTemplateTestCase();
+
     /**
      * Ref and Unref - only here for testing of Ptr<>
      *
      * @{
      */
-    void Ref() const {};
-    void Unref() const {};
+    void Ref() const
+    {
+    }
+
+    void Unref() const
+    {
+    }
+
     /** @} */
 
   private:

src/internet/test/ipv6-list-routing-test-suite.cc

@@ -76,7 +76,9 @@ class Ipv6ARouting : public Ipv6RoutingProtocol
     {
     }
 
-    void PrintRoutingTable(Ptr<OutputStreamWrapper> stream, Time::Unit unit) const override{};
+    void PrintRoutingTable(Ptr<OutputStreamWrapper> stream, Time::Unit unit) const override
+    {
+    }
 };
 
 /**
@@ -142,7 +144,9 @@ class Ipv6BRouting : public Ipv6RoutingProtocol
     {
     }
 
-    void PrintRoutingTable(Ptr<OutputStreamWrapper> stream, Time::Unit unit) const override{};
+    void PrintRoutingTable(Ptr<OutputStreamWrapper> stream, Time::Unit unit) const override
+    {
+    }
 };
 
 /**

src/internet/test/tcp-advertised-window-test.cc

@@ -226,7 +226,10 @@ class TcpDropRatioErrorModel : public TcpGeneralErrorModel
                     uint32_t packetSize) override;
 
   private:
-    void DoReset() override{};
+    void DoReset() override
+    {
+    }
+
     double m_dropRatio;                //!< Drop ratio
     Ptr<UniformRandomVariable> m_prng; //!< Random variable
 };

src/mesh/model/dot11s/hwmp-protocol-mac.h

@@ -50,8 +50,12 @@ class HwmpProtocolMac : public MeshWifiInterfaceMacPlugin
                               WifiMacHeader& header,
                               Mac48Address from,
                               Mac48Address to) override;
+
     // Update beacon is empty, because HWMP does not know anything about beacons
-    void UpdateBeacon(MeshWifiBeacon& beacon) const override{};
+    void UpdateBeacon(MeshWifiBeacon& beacon) const override
+    {
+    }
+
     int64_t AssignStreams(int64_t stream) override;
 
   private:

src/mesh/model/flame/flame-protocol-mac.h

@@ -59,11 +59,14 @@ class FlameProtocolMac : public MeshWifiInterfaceMacPlugin
                               WifiMacHeader& header,
                               Mac48Address from,
                               Mac48Address to) override;
+
     /**
      * Update beacon is empty, because FLAME does not know anything about beacons
      * \param beacon the beacon
      */
-    void UpdateBeacon(MeshWifiBeacon& beacon) const override{};
+    void UpdateBeacon(MeshWifiBeacon& beacon) const override
+    {
+    }
 
     /**
      * AssignStreams is empty, because this model doesn't use random variables

src/mesh/model/mesh-wifi-interface-mac-plugin.h

@@ -31,7 +31,10 @@ class MeshWifiInterfaceMacPlugin : public SimpleRefCount<MeshWifiInterfaceMacPlu
 {
   public:
     /// This is for subclasses
-    virtual ~MeshWifiInterfaceMacPlugin(){};
+    virtual ~MeshWifiInterfaceMacPlugin()
+    {
+    }
+
     /**
      * Each plugin must be installed on an interface to work
      *

src/mesh/test/mesh-information-element-vector-test-suite.cc

@@ -36,7 +36,10 @@ using namespace ns3;
 struct MeshInformationElementVectorBist : public TestCase
 {
     MeshInformationElementVectorBist()
-        : TestCase("Serialization test for all mesh information elements"){};
+        : TestCase("Serialization test for all mesh information elements")
+    {
+    }
+
     void DoRun() override;
 };
 

src/mobility/helper/ns2-mobility-helper.cc

@@ -82,12 +82,15 @@ struct DestinationPoint
     EventId m_stopEvent;    //!< Event scheduling node's stop. May be canceled if needed.
     double m_travelStartTime;   //!< Travel start time is needed to calculate actually traveled time
     double m_targetArrivalTime; //!< When a station arrives to a destination
+
     DestinationPoint()
         : m_startPosition(Vector(0, 0, 0)),
           m_speed(Vector(0, 0, 0)),
           m_finalPosition(Vector(0, 0, 0)),
           m_travelStartTime(0),
-          m_targetArrivalTime(0){};
+          m_targetArrivalTime(0)
+    {
+    }
 };
 
 /**

src/propagation/model/propagation-cache.h

@@ -24,8 +24,13 @@ template <class T>
 class PropagationCache
 {
   public:
-    PropagationCache(){};
-    ~PropagationCache(){};
+    PropagationCache()
+    {
+    }
+
+    ~PropagationCache()
+    {
+    }
 
     /**
      * Get the model associated with the path
@@ -89,7 +94,10 @@ class PropagationCache
                                   uint32_t modelUid)
             : m_srcMobility(a),
               m_dstMobility(b),
-              m_spectrumModelUid(modelUid){};
+              m_spectrumModelUid(modelUid)
+        {
+        }
+
         Ptr<const MobilityModel> m_srcMobility; //!< 1st node mobility model
         Ptr<const MobilityModel> m_dstMobility; //!< 2nd node mobility model
         uint32_t m_spectrumModelUid;            //!< model UID

src/test/traced/traced-callback-typedef-test-suite.cc

@@ -268,8 +268,13 @@ class TracedCallbackTypedefTestCase::Checker : public Object
     TracedCallback<Ts...> m_cb;
 
   public:
-    Checker(){};
-    ~Checker() override{};
+    Checker()
+    {
+    }
+
+    ~Checker() override
+    {
+    }
 
     /// Arguments of the TracedCallback.
     std::tuple<std::remove_pointer_t<std::remove_cvref_t<Ts>>...> m_items;

src/wifi/model/wifi-remote-station-manager.h

@@ -57,7 +57,10 @@ struct RxSignalInfo;
  */
 struct WifiRemoteStation
 {
-    virtual ~WifiRemoteStation(){};
+    virtual ~WifiRemoteStation()
+    {
+    }
+
     WifiRemoteStationState* m_state; //!< Remote station state
     std::pair<dBm_u, Time>
         m_rssiAndUpdateTimePair; //!< RSSI of the most recent packet received from