Merge pull request #221 from lars-petter-hauge/warn_unused_sumkeys

WIP:Warn unused sumkeys
OPM · Dec 1, 2017 · 2e047d2 · 2e047d2
2 parents 96c14c9 + e9f9d74
commit 2e047d2
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/opm/output/eclipse/Summary.cpp b/opm/output/eclipse/Summary.cpp
@@ -867,7 +867,7 @@ Summary::Summary( const EclipseState& st,
         if (init_config.getRestartRootName().size() <= ECL_STRING8_LENGTH * SUMMARY_RESTART_SIZE)
             restart_case = init_config.getRestartRootName().c_str();
         else
-            OpmLog::warning("Resart case too long - not embedded in SMSPEC file");
+            OpmLog::warning("Restart case too long - not embedded in SMSPEC file");
     }
     ecl_sum.reset( ecl_sum_alloc_restart_writer(basename,
                                                 restart_case,
@@ -883,6 +883,8 @@ Summary::Summary( const EclipseState& st,
     /* register all keywords handlers and pair with the newly-registered ert
      * entry.
      */
+    std::set< std::string > unsupported_keywords;
+
     for( const auto& node : sum ) {
         const auto* keyword = node.keyword();
 
@@ -905,7 +907,10 @@ Summary::Summary( const EclipseState& st,
 
 	    this->handlers->misc_nodes.emplace( keyword, nodeptr ); 
         } else {
-	    if( funs.find( keyword ) == funs.end() ) continue;
+	        if( funs.find( keyword ) == funs.end() ) {
+                unsupported_keywords.insert(keyword);
+                continue;
+            }
 
             if ((node.type() == ECL_SMSPEC_COMPLETION_VAR) || (node.type() == ECL_SMSPEC_BLOCK_VAR)) {
                 int global_index = node.num() - 1;
@@ -940,6 +945,9 @@ Summary::Summary( const EclipseState& st,
 	    this->handlers->handlers.emplace_back( nodeptr, handle );
 	}
     }
+    for ( const auto& keyword : unsupported_keywords ) {
+        Opm::OpmLog::info("Keyword " + std::string(keyword) + " is unhandled");
+    }
 }
 
 void Summary::add_timestep( int report_step,