Only warn about unknown tags in instrumentation.xml

Richard J. Senior:
Without this commit, unknown top-level tags in instrumentation.xml are treated
as errors and the rest of the file is not processed.

Users running Flightgear versions prior to 3.2 have experienced problems with
the comm-radio tags after downloading updated versions of aircraft from FGADDON.
Instrumentation listed after the unrecognized comm-radio tags is not processed
for these users and is inactive in the cockpit.

This commit changes the instrumentation build method so that unrecognized tags
are treated as warnings. This won't help users running older versions but
protects against the same problems occurring if new tags are added to
instrumentation in the future.

diff --git a/src/Instrumentation/instrument_mgr.cxx b/src/Instrumentation/instrument_mgr.cxx
index 5b56f978016b7dbcf22a9d5c344e58b0e3694be3..6aebd16c61bbca8824bc85607cfe229045c1a57c 100644 (file)
--- a/src/Instrumentation/instrument_mgr.cxx
+++ b/src/Instrumentation/instrument_mgr.cxx
@@ -222,9 +222,8 @@ bool FGInstrumentMgr::build (SGPropertyNode* config_props)
             set_subsystem( id, new TCAS( node ), 0.2);
             
         } else {
-            SG_LOG( SG_INSTR, SG_ALERT, "Unknown top level section: "
-                    << name );
-            return false;
+            SG_LOG( SG_INSTR, SG_ALERT,
+                     "Unknown top level section in instrumentation: " << name );
         }
       
       // only push to our array if we actually built an insturment