Update BMI C++ header to match upstream with virtual dtor and correct…

… namespace
NOAA-OWP · Jun 11, 2024 · 50289e9 · 50289e9
1 parent 8ae1d74
commit 50289e9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/bmi/bmi.hxx b/bmi/bmi.hxx
@@ -6,13 +6,14 @@
 
 #ifndef BMI_HXX
 #define BMI_HXX
+
 #include <string>
 #include <vector>
 
-namespace bmixx {
+namespace bmi {
 
-  //const int BMI_SUCCESS = 0;
-  //  const int BMI_FAILURE = 1;
+  const int BMI_SUCCESS = 0;
+  const int BMI_FAILURE = 1;
 
   const int MAX_COMPONENT_NAME = 2048;
   const int MAX_VAR_NAME = 2048;
@@ -21,6 +22,8 @@ namespace bmixx {
 
   class Bmi {
     public:
+      virtual ~Bmi() { }
+
       // Model control functions.
       virtual void Initialize(std::string config_file) = 0;
       virtual void Update() = 0;

diff --git a/include/bmi_soil_moisture_profile.hxx b/include/bmi_soil_moisture_profile.hxx
@@ -16,7 +16,7 @@ namespace coupler {
 
 }
 
-class BmiSoilMoistureProfile : public bmixx::Bmi {
+class BmiSoilMoistureProfile : public bmi::Bmi {
 public:
   BmiSoilMoistureProfile() {
     this->input_var_names[0]  = "soil_storage";