MultiFabRegister: throw in get

Throw a runtime exception instead of returning a `nullptr` if a field is requested via the getter.
ECP-WarpX · Oct 2, 2024 · 2c2f48c · 2c2f48c
1 parent 025f70e
commit 2c2f48c
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/Source/ablastr/fields/MultiFabRegister.cpp b/Source/ablastr/fields/MultiFabRegister.cpp
@@ -350,8 +350,7 @@ namespace ablastr::fields
     )
     {
         if (m_mf_register.count(internal_name) == 0) {
-            // FIXME: temporary, throw a std::runtime_error
-            // throw std::runtime_error("MultiFabRegister::get name does not exist in register: " + key);
+            throw std::runtime_error("MultiFabRegister::get name does not exist in register: " + internal_name);
             return nullptr;
         }
         amrex::MultiFab & mf = m_mf_register.at(internal_name).m_mf;
@@ -365,8 +364,7 @@ namespace ablastr::fields
     ) const
     {
         if (m_mf_register.count(internal_name) == 0) {
-            // FIXME: temporary, throw a std::runtime_error
-            // throw std::runtime_error("MultiFabRegister::get name does not exist in register: " + internal_name);
+            throw std::runtime_error("MultiFabRegister::get name does not exist in register: " + internal_name);
             return nullptr;
         }
         amrex::MultiFab const & mf = m_mf_register.at(internal_name).m_mf;