Commits on Oct 12, 2023

1. Fix for SST_ELI_DECLARE_NEW_BASE when BuilderInfo is a dependent … …

   …name.
   
   This is easier to describe with an example first. Imagine the following code:
   
   ```c++
   template <class Component>
   struct MyComponentBase : public Component {
       SST_ELI_DECLARE_NEW_BASE(Component, MyComponentBase);
       // tools that help us with SST::Component interfaces
   };
   
   struct MyComponent : MyComponentBase<SST::Component> {
       // A custom component
   };
   
   struct MyRouter : MyComponentBase<SST::Merlin::Router> {
       // A merlin router implementation
   };
   ```
   
   In this context, inside the `SST_ELI_DECLARE_NEW_BASE`, the `BuilderInfo` is a
   dependent name in the `addDerivedInfo` template function, and can't be found
   without qualification. The qualified name is `__LocalEliBase::BuilderInfo` and
   it needs a `typename` pre-C++20.
   
   This patch adds that typename to support this template pattern. This pattern
   allows us to write utility code that can be injected into _any_ component
   hierarchy.

   

   ldalessa committed Oct 12, 2023
   Configuration menu

   • View commit details
   • Copy full SHA for 5432cca
   • Browse repository at this point

   Copy the full SHA

   5432cca View commit details

   Browse the repository at this point in the history