Inline trivial default constructors and destructors

src/sst/core/baseComponent.cc

@@ -36,8 +36,6 @@
 
 namespace SST {
 
-BaseComponent::BaseComponent() : SST::Core::Serialization::serializable_base() {}
-
 BaseComponent::BaseComponent(ComponentId_t id) :
     SST::Core::Serialization::serializable_base(),
     my_info(Simulation_impl::getSimulation()->getComponentInfo(id)),

src/sst/core/baseComponent.h

@@ -70,8 +70,7 @@ class BaseComponent : public SST::Core::Serialization::serializable_base
         BaseComponent*, Statistics::StatisticProcessingEngine*, const std::string& /*type*/,
         const std::string& /*name*/, const std::string& /*subId*/, Params&)>;
 
-    // For serialization only
-    BaseComponent();
+    BaseComponent() = default; // For serialization only
 
 public:
     BaseComponent(ComponentId_t id);

src/sst/core/checkpointAction.cc

@@ -124,8 +124,6 @@ CheckpointAction::CheckpointAction(Config* cfg, RankInfo this_rank, Simulation_i
     }
 }
 
-CheckpointAction::~CheckpointAction() {}
-
 void
 CheckpointAction::insertIntoTimeVortex(Simulation_impl* sim)
 {

src/sst/core/checkpointAction.h

@@ -64,7 +64,7 @@ class CheckpointAction : public Action
     Create a new checkpoint object for the simulation core to initiate checkpoints
     */
     CheckpointAction(Config* cfg, RankInfo this_rank, Simulation_impl* sim, TimeConverter* period);
-    ~CheckpointAction();
+    ~CheckpointAction() = default;
 
     /**
        Indicates CheckpointAction should be inserted into the

src/sst/core/component.cc

@@ -30,8 +30,6 @@ Component::Component(ComponentId_t id) : BaseComponent(id)
     // currentlyLoadingSubComponent = my_info;
 }
 
-Component::~Component() {}
-
 void
 Component::registerAsPrimaryComponent()
 {
@@ -59,7 +57,4 @@ Component::serialize_order(SST::Core::Serialization::serializer& ser)
     BaseComponent::serialize_order(ser);
 }
 
-// For serialization only
-Component::Component() : BaseComponent() {}
-
 } // namespace SST

src/sst/core/component.h

@@ -48,7 +48,7 @@ class Component : public BaseComponent
         @param id Unique component ID
     */
     Component(ComponentId_t id);
-    virtual ~Component();
+    virtual ~Component() override = default;
 
     /** Register as a primary component, which allows the component to
         specify when it is and is not OK to end simulation.  The
@@ -97,9 +97,7 @@ class Component : public BaseComponent
 
 protected:
     friend class SubComponent;
-
-    // For Serialization only
-    Component();
+    Component() = default; // For Serialization only
 };
 
 } // namespace SST

src/sst/core/componentExtension.cc

@@ -26,7 +26,4 @@ ComponentExtension::serialize_order(SST::Core::Serialization::serializer& ser)
     BaseComponent::serialize_order(ser);
 }
 
-// For serialization only
-ComponentExtension::ComponentExtension() : BaseComponent() {}
-
 } // namespace SST

src/sst/core/componentExtension.h

@@ -31,11 +31,10 @@ class ComponentExtension : public BaseComponent
 public:
     ComponentExtension(ComponentId_t id);
 
-    virtual ~ComponentExtension() {};
+    virtual ~ComponentExtension() override = default;
 
 private:
-    // For serialization only
-    ComponentExtension();
+    ComponentExtension() = default; // For serialization only
 
     ImplementSerializable(SST::ComponentExtension)
     void serialize_order(SST::Core::Serialization::serializer& ser) override;

src/sst/core/event.cc

@@ -23,8 +23,6 @@ namespace SST {
 std::atomic<uint64_t>     SST::Event::id_counter(0);
 const SST::Event::id_type SST::Event::NO_ID = std::make_pair(0, -1);
 
-Event::~Event() {}
-
 void
 Event::execute()
 {

src/sst/core/event.h

@@ -79,7 +79,7 @@ class Event : public Activity
         last_comp  = "";
 #endif
     }
-    virtual ~Event();
+    ~Event() override = default;
 
     /** Clones the event in for the case of a broadcast */
     virtual Event* clone();

src/sst/core/heartbeat.cc

@@ -36,8 +36,6 @@ SimulatorHeartbeat::SimulatorHeartbeat(
     if ( (0 == this_rank) ) { lastTime = sst_get_cpu_time(); }
 }
 
-SimulatorHeartbeat::~SimulatorHeartbeat() {}
-
 void
 SimulatorHeartbeat::schedule()
 {

src/sst/core/heartbeat.h

@@ -36,7 +36,7 @@ class SimulatorHeartbeat : public Action
     Create a new heartbeat object for the simulation core to show progress
     */
     SimulatorHeartbeat(Config* cfg, int this_rank, Simulation_impl* sim, TimeConverter* period);
-    ~SimulatorHeartbeat();
+    ~SimulatorHeartbeat() = default;
 
     // Used to re-schedule the new heartbeat event during restart
     void schedule();

src/sst/core/initQueue.cc

@@ -15,7 +15,6 @@
 
 namespace SST {
 
-InitQueue::InitQueue() : ActivityQueue() {}
 InitQueue::~InitQueue()
 {
     // Need to delete any events left in the queue

src/sst/core/initQueue.h

@@ -24,7 +24,7 @@ namespace SST {
 class InitQueue : public ActivityQueue
 {
 public:
-    InitQueue();
+    InitQueue() = default;
     ~InitQueue();
 
     bool      empty() override;

src/sst/core/interactiveConsole.cc

@@ -25,11 +25,6 @@ namespace SST {
 SST_ELI_DEFINE_INFO_EXTERN(SST::InteractiveConsole)
 SST_ELI_DEFINE_CTOR_EXTERN(SST::InteractiveConsole)
 
-
-InteractiveConsole::InteractiveConsole() {}
-
-InteractiveConsole::~InteractiveConsole() {}
-
 UnitAlgebra
 InteractiveConsole::getCoreTimeBase() const
 {

src/sst/core/interactiveConsole.h

@@ -53,8 +53,8 @@ class InteractiveConsole
     /**
     Create a new checkpoint object for the simulation core to initiate checkpoints
     */
-    InteractiveConsole();
-    virtual ~InteractiveConsole();
+    InteractiveConsole()          = default;
+    virtual ~InteractiveConsole() = default;
 
     /** Called by TimeVortex to trigger checkpoint on simulation clock interval - not used in parallel simulation */
     virtual void execute(const std::string& msg) = 0;

src/sst/core/model/python/pymodel_comp.h

@@ -54,13 +54,13 @@ struct PyComponent : ComponentHolder
     uint16_t subCompId;
 
     PyComponent(ComponentPy_t* pobj, SST::ComponentId_t id) : ComponentHolder(pobj, id), subCompId(0) {}
-    ~PyComponent() {}
+    ~PyComponent() override = default;
 };
 
 struct PySubComponent : ComponentHolder
 {
     PySubComponent(ComponentPy_t* pobj, SST::ComponentId_t id) : ComponentHolder(pobj, id) {}
-    ~PySubComponent() {}
+    ~PySubComponent() override = default;
     int getSlot();
 };
 

src/sst/core/pollingLinkQueue.cc

@@ -15,7 +15,6 @@
 
 namespace SST {
 
-PollingLinkQueue::PollingLinkQueue() : ActivityQueue() {}
 PollingLinkQueue::~PollingLinkQueue()
 {
     // Need to delete any events left in the queue

src/sst/core/pollingLinkQueue.h

@@ -24,7 +24,7 @@ namespace SST {
 class PollingLinkQueue : public ActivityQueue
 {
 public:
-    PollingLinkQueue();
+    PollingLinkQueue() = default;
     ~PollingLinkQueue();
 
     bool      empty() override;

src/sst/core/portModule.cc

@@ -21,8 +21,6 @@ namespace SST {
 SST_ELI_DEFINE_INFO_EXTERN(PortModule)
 SST_ELI_DEFINE_CTOR_EXTERN(PortModule)
 
-PortModule::PortModule() {}
-
 uintptr_t
 PortModule::registerLinkAttachTool(const AttachPointMetaData& UNUSED(mdata))
 {

src/sst/core/portModule.h

@@ -56,8 +56,7 @@ class PortModule :
         // ELI::ProvidesStats, // Will add stats in the future
         ELI::ProvidesAttributes)
 
-
-    PortModule();
+    PortModule() = default;
     virtual ~PortModule() {};
 
     /******* Functions inherited from Link::AttachPoint *******/

src/sst/core/realtime.cc

@@ -82,8 +82,6 @@ SST_ELI_DEFINE_INFO_EXTERN(RealTimeAction)
 SST_ELI_DEFINE_CTOR_EXTERN(RealTimeAction)
 
 
-RealTimeAction::RealTimeAction() {}
-
 UnitAlgebra
 RealTimeAction::getCoreTimeBase() const
 {

src/sst/core/realtimeAction.h

@@ -32,8 +32,8 @@ class RealTimeAction
     SST_ELI_DECLARE_DEFAULT_INFO_EXTERN()
     SST_ELI_DECLARE_DEFAULT_CTOR_EXTERN()
 
+    RealTimeAction() = default;
 
-    RealTimeAction();
     virtual ~RealTimeAction() = default;
 
     /* Optional function called just before run loop starts. Passes in

src/sst/core/sstinfo.cc

@@ -753,8 +753,6 @@ SSTInfoConfig::SSTInfoConfig(bool suppress_print) : ConfigShared(suppress_print,
     addPositionalCallback(std::bind(&SSTInfoConfig::setPositionalArg, this, _1, _2));
 }
 
-SSTInfoConfig::~SSTInfoConfig() {}
-
 std::string
 SSTInfoConfig::getUsagePrelude()
 {

src/sst/core/sstinfo.h

@@ -42,7 +42,7 @@ class SSTInfoConfig : public ConfigShared
     using FilterMap_t = std::multimap<std::string, std::string>;
     /** Create a new SSTInfo configuration and parse the Command Line. */
     SSTInfoConfig(bool suppress_print);
-    ~SSTInfoConfig();
+    ~SSTInfoConfig() override = default;
 
     /** Return the list of elements to be processed. */
     std::set<std::string> getElementsToProcessArray()

src/sst/core/statapi/statoutput.cc

@@ -33,8 +33,6 @@ StatisticOutput::StatisticOutput(Params& outputParameters)
 SST_ELI_DEFINE_CTOR_EXTERN(StatisticOutput)
 SST_ELI_DEFINE_INFO_EXTERN(StatisticOutput)
 
-StatisticOutput::~StatisticOutput() {}
-
 void
 StatisticOutput::serialize_order(SST::Core::Serialization::serializer& ser)
 {

src/sst/core/statapi/statoutput.h

@@ -60,7 +60,7 @@ class StatisticOutput : public SST::Core::Serialization::serializable
     using FieldNameMap_t   = std::unordered_map<std::string, fieldHandle_t>;
 
 public:
-    ~StatisticOutput();
+    ~StatisticOutput() = default;
 
     /** Return the Statistic Output name */
     std::string& getStatisticOutputName() { return m_statOutputName; }

src/sst/core/subcomponent.cc

@@ -28,7 +28,4 @@ SubComponent::serialize_order(SST::Core::Serialization::serializer& ser)
     BaseComponent::serialize_order(ser);
 }
 
-// For serialization only
-SubComponent::SubComponent() : BaseComponent() {}
-
 } // namespace SST

src/sst/core/subcomponent.h

@@ -58,8 +58,7 @@ class SubComponent : public BaseComponent
     virtual void finish() override {}
 
 protected:
-    // For serialization only
-    SubComponent();
+    SubComponent() = default; // For serialization only
     void serialize_order(SST::Core::Serialization::serializer& ser) override;
 
     friend class Component;

src/sst/core/sync/syncManager.cc

@@ -321,8 +321,6 @@ SyncManager::SyncManager()
     sim_ = Simulation_impl::getSimulation();
 }
 
-SyncManager::~SyncManager() {}
-
 /** Register a Link which this Sync Object is responsible for */
 ActivityQueue*
 SyncManager::registerLink(const RankInfo& to_rank, const RankInfo& from_rank, const std::string& name, Link* link)

src/sst/core/sync/syncManager.h

@@ -134,7 +134,7 @@ class SyncManager : public Action
         const RankInfo& rank, const RankInfo& num_ranks, SimTime_t min_part,
         const std::vector<SimTime_t>& interThreadLatencies, RealTimeManager* real_time);
     SyncManager(); // For serialization only
-    virtual ~SyncManager();
+    virtual ~SyncManager() = default;
 
     /** Register a Link which this Sync Object is responsible for */
     ActivityQueue*

src/sst/core/sync/syncQueue.cc

@@ -34,8 +34,6 @@ using namespace Core::Serialization;
 
 RankSyncQueue::RankSyncQueue(RankInfo to_rank) : SyncQueue(to_rank), buffer(nullptr), buf_size(0) {}
 
-RankSyncQueue::~RankSyncQueue() {}
-
 bool
 RankSyncQueue::empty()
 {

src/sst/core/sync/syncQueue.h

@@ -58,7 +58,7 @@ class RankSyncQueue : public SyncQueue
     };
 
     RankSyncQueue(RankInfo to_rank);
-    ~RankSyncQueue();
+    ~RankSyncQueue() = default;
 
     bool      empty() override;
     int       size() override;

src/sst/core/testElements/coreTest_Checkpoint.h

@@ -102,7 +102,7 @@ class coreTestCheckpoint : public SST::Component
     )
 
     coreTestCheckpoint(ComponentId_t id, SST::Params& params);
-    virtual ~coreTestCheckpoint();
+    ~coreTestCheckpoint();
 
     void init(unsigned phase) override;
 

src/sst/core/testElements/coreTest_Component.cc

@@ -73,11 +73,6 @@ coreTestComponent::~coreTestComponent()
     delete rng;
 }
 
-coreTestComponent::coreTestComponent() : coreTestComponentBase2()
-{
-    // for serialization only
-}
-
 // incoming events are scanned and deleted
 void
 coreTestComponent::handleEvent(Event* ev)

src/sst/core/testElements/coreTest_Component.h

@@ -120,7 +120,7 @@ class coreTestComponent : public coreTestComponentBase2
 
     void serialize_order(SST::Core::Serialization::serializer& ser) override;
     ImplementSerializable(SST::CoreTestComponent::coreTestComponent)
-    coreTestComponent(); // for serialization only
+    coreTestComponent() = default; // for serialization only
 
 private:
     coreTestComponent(const coreTestComponent&) = delete;            // do not implement

src/sst/core/testElements/coreTest_Links.cc

@@ -55,8 +55,6 @@ coreTestLinks::coreTestLinks(ComponentId_t id, Params& params) : Component(id),
     registerClock("100 MHz", new Clock::Handler<coreTestLinks>(this, &coreTestLinks::clockTic));
 }
 
-coreTestLinks::~coreTestLinks() {}
-
 // incoming events are scanned and deleted
 void
 coreTestLinks::handleEvent(Event* ev, std::string from)

src/sst/core/testElements/coreTest_Links.h

@@ -51,7 +51,7 @@ class coreTestLinks : public SST::Component
     )
 
     coreTestLinks(SST::ComponentId_t id, SST::Params& params);
-    ~coreTestLinks();
+    ~coreTestLinks() = default;
 
     void setup() {}
     void finish() {}

src/sst/core/testElements/coreTest_Module.cc

@@ -126,10 +126,6 @@ void
 coreTestModuleLoader::finish()
 {}
 
-coreTestModuleLoader::coreTestModuleLoader() : Component()
-{ /* For serialization ONLY*/
-}
-
 bool coreTestModuleLoader::tick(SST::Cycle_t)
 {
     uint32_t next = rng_module->getNext();