]> git.mxchange.org Git - flightgear.git/blobdiff - src/Cockpit/panel_io.cxx
Use !strcmp for all property string-value comparisons.
[flightgear.git] / src / Cockpit / panel_io.cxx
index f6ba1bed13ba13c48128f6b869460192c1fc649e..b81717430377f78988ba13e832d70cb0bbbd1cb3 100644 (file)
@@ -27,6 +27,7 @@
 #endif
 
 #include <simgear/compiler.h>
+#include <simgear/misc/exception.hxx>
 
 #include <simgear/misc/sg_path.hxx>
 #include <simgear/debug/logstream.hxx>
@@ -39,6 +40,8 @@
 #include <Main/globals.hxx>
 #include <Main/fg_props.hxx>
 
+#include <GUI/gui.h>
+
 #include "panel.hxx"
 #include "steam.hxx"
 #include "panel_io.hxx"
@@ -50,53 +53,6 @@ SG_USING_STD(ifstream);
 SG_USING_STD(string);
 
 
-\f
-////////////////////////////////////////////////////////////////////////
-// Default panel, instrument, and layer for when things go wrong...
-////////////////////////////////////////////////////////////////////////
-
-static FGCroppedTexture defaultTexture("Textures/default.rgb");
-
-
-/**
- * Default layer: the default texture.
- */
-class DefaultLayer : public FGTexturedLayer
-{
-public:
-  DefaultLayer () : FGTexturedLayer(defaultTexture)
-  {
-  }
-  
-};
-
-/**
- * Default instrument: a single default layer.
- */
-class DefaultInstrument : public FGLayeredInstrument
-{
-public:
-  DefaultInstrument (int x, int y, int w, int h)
-    : FGLayeredInstrument(x, y, w, h)
-  {
-    addLayer(new DefaultLayer());
-  }
-};
-
-
-/**
- * Default panel: the default texture.
- */
-class DefaultPanel : public FGPanel
-{
-public:
-  DefaultPanel (int x, int y, int w, int h) : FGPanel(x, y, w, h)
-  {
-    setBackground(defaultTexture.getTexture());
-  }
-};
-
-
 \f
 ////////////////////////////////////////////////////////////////////////
 // Built-in layer for the magnetic compass ribbon layer.
@@ -117,7 +73,7 @@ public:
 FGMagRibbon::FGMagRibbon (int w, int h)
   : FGTexturedLayer(w, h)
 {
-  FGCroppedTexture texture("Aircraft/c172/Instruments/Textures/compass-ribbon.rgb");
+  FGCroppedTexture texture("Aircraft/Instruments/Textures/compass-ribbon.rgb");
   setTexture(texture);
 }
 
@@ -218,13 +174,31 @@ FGMagRibbon::draw ()
 static FGCroppedTexture
 readTexture (const SGPropertyNode * node)
 {
-  FGCroppedTexture texture(node->getStringValue("path"),
-                          node->getFloatValue("x1"),
-                          node->getFloatValue("y1"),
-                          node->getFloatValue("x2", 1.0),
-                          node->getFloatValue("y2", 1.0));
-  SG_LOG(SG_INPUT, SG_INFO, "Read texture " << node->getName());
-  return texture;
+    FGCroppedTexture texture(node->getStringValue("path"),
+                            node->getFloatValue("x1"),
+                            node->getFloatValue("y1"),
+                            node->getFloatValue("x2", 1.0),
+                            node->getFloatValue("y2", 1.0));
+    SG_LOG(SG_COCKPIT, SG_DEBUG, "Read texture " << node->getName());
+    return texture;
+}
+
+
+/**
+ * Test for a condition in the current node.
+ */
+\f
+////////////////////////////////////////////////////////////////////////
+// Read a condition and use it if necessary.
+////////////////////////////////////////////////////////////////////////
+
+static void
+readConditions (FGConditional * component, const SGPropertyNode * node)
+{
+  const SGPropertyNode * conditionNode = node->getChild("condition");
+  if (conditionNode != 0)
+                               // The top level is implicitly AND
+    component->setCondition(fgReadCondition(conditionNode));
 }
 
 
@@ -232,8 +206,9 @@ readTexture (const SGPropertyNode * node)
  * Read an action from the instrument's property list.
  *
  * The action will be performed when the user clicks a mouse button
- * within the specified region of the instrument.  Actions always
- * work by modifying the value of a property (see the SGValue class).
+ * within the specified region of the instrument.  Actions always work
+ * by modifying the value of a property (see the SGPropertyNode
+ * class).
  *
  * The following action types are defined:
  *
@@ -256,10 +231,7 @@ readTexture (const SGPropertyNode * node)
 static FGPanelAction *
 readAction (const SGPropertyNode * node, float w_scale, float h_scale)
 {
-  FGPanelAction * action = 0;
-
   string name = node->getStringValue("name");
-  string type = node->getStringValue("type");
 
   int button = node->getIntValue("button");
   int x = int(node->getIntValue("x") * w_scale);
@@ -267,49 +239,16 @@ readAction (const SGPropertyNode * node, float w_scale, float h_scale)
   int w = int(node->getIntValue("w") * w_scale);
   int h = int(node->getIntValue("h") * h_scale);
 
-  if (type == "") {
-    SG_LOG(SG_INPUT, SG_ALERT,
-          "No type supplied for action " << name << " assuming \"adjust\"");
-    type = "adjust";
-  }
-
-                               // Adjust a property value
-  if (type == "adjust") {
-    string propName = node->getStringValue("property");
-    SGValue * value = fgGetValue(propName, true);
-    float increment = node->getFloatValue("increment", 1.0);
-    float min = node->getFloatValue("min", 0.0);
-    float max = node->getFloatValue("max", 0.0);
-    bool wrap = node->getBoolValue("wrap", false);
-    if (min == max)
-      SG_LOG(SG_INPUT, SG_ALERT, "Action " << node->getName()
-            << " has same min and max value");
-    action = new FGAdjustAction(button, x, y, w, h, value,
-                               increment, min, max, wrap);
-  } 
-
-                               // Swap two property values
-  else if (type == "swap") {
-    string propName1 = node->getStringValue("property1");
-    string propName2 = node->getStringValue("property2");
-    SGValue * value1 = fgGetValue(propName1, true);
-    SGValue * value2 = fgGetValue(propName2, true);
-    action = new FGSwapAction(button, x, y, w, h, value1, value2);
-  } 
+  FGPanelAction * action = new FGPanelAction(button, x, y, w, h);
 
-                               // Toggle a boolean value
-  else if (type == "toggle") {
-    string propName = node->getStringValue("property");
-    SGValue * value = fgGetValue(propName, true);
-    action = new FGToggleAction(button, x, y, w, h, value);
-  } 
-
-                               // Unrecognized type
-  else {
-    SG_LOG(SG_INPUT, SG_ALERT, "Unrecognized action type " << type);
-    return 0;
+  vector<const SGPropertyNode *>bindings = node->getChildren("binding");
+  for (unsigned int i = 0; i < bindings.size(); i++) {
+    SG_LOG(SG_INPUT, SG_INFO, "Reading binding "
+          << bindings[i]->getStringValue("command"));
+    action->addBinding(new FGBinding(bindings[i])); // TODO: allow modifiers
   }
 
+  readConditions(action, node);
   return action;
 }
 
@@ -347,20 +286,20 @@ readTransformation (const SGPropertyNode * node, float w_scale, float h_scale)
   string name = node->getName();
   string type = node->getStringValue("type");
   string propName = node->getStringValue("property", "");
-  SGValue * value = 0;
+  SGPropertyNode * target = 0;
 
   if (type == "") {
-    SG_LOG(SG_INPUT, SG_ALERT,
-          "No type supplied for transformation " << name
-          << " assuming \"rotation\"");
+    SG_LOG( SG_COCKPIT, SG_ALERT,
+            "No type supplied for transformation " << name
+            << " assuming \"rotation\"" );
     type = "rotation";
   }
 
-  if (propName != (string)"") {
-    value = fgGetValue(propName, true);
+  if (!propName.empty()) {
+    target = fgGetNode(propName.c_str(), true);
   }
 
-  t->value = value;
+  t->node = target;
   t->min = node->getFloatValue("min", -9999999);
   t->max = node->getFloatValue("max", 99999999);
   t->factor = node->getFloatValue("scale", 1.0);
@@ -369,18 +308,20 @@ readTransformation (const SGPropertyNode * node, float w_scale, float h_scale)
                                // Check for an interpolation table
   const SGPropertyNode * trans_table = node->getNode("interpolation");
   if (trans_table != 0) {
-    cerr << "Found interpolation table with " << trans_table->nChildren() << "children" << endl;
+    SG_LOG( SG_COCKPIT, SG_INFO, "Found interpolation table with "
+            << trans_table->nChildren() << "children" );
     t->table = new SGInterpTable();
     for(int i = 0; i < trans_table->nChildren(); i++) {
       const SGPropertyNode * node = trans_table->getChild(i);
-      if (node->getName() == "entry") {
+      if (string(node->getName()) == "entry") {
        double ind = node->getDoubleValue("ind", 0.0);
        double dep = node->getDoubleValue("dep", 0.0);
-       cerr << "Adding interpolation entry " << ind << "==>" << dep << endl;
+       SG_LOG( SG_COCKPIT, SG_INFO, "Adding interpolation entry "
+                << ind << "==>" << dep );
        t->table->addEntry(ind, dep);
       } else {
-       SG_LOG(SG_INPUT, SG_INFO, "Skipping " << node->getName()
-              << " in interpolation");
+       SG_LOG( SG_COCKPIT, SG_INFO, "Skipping " << node->getName()
+                << " in interpolation" );
       }
     }
   } else {
@@ -413,12 +354,13 @@ readTransformation (const SGPropertyNode * node, float w_scale, float h_scale)
   } 
 
   else {
-    SG_LOG(SG_INPUT, SG_ALERT, "Unrecognized transformation type " << type);
+    SG_LOG( SG_COCKPIT, SG_ALERT, "Unrecognized transformation type " << type );
     delete t;
     return 0;
   }
 
-  SG_LOG(SG_INPUT, SG_INFO, "Read transformation " << name);
+  readConditions(t, node);
+  SG_LOG( SG_COCKPIT, SG_DEBUG, "Read transformation " << name );
   return t;
 }
 
@@ -449,8 +391,8 @@ readTextChunk (const SGPropertyNode * node)
 
                                // Default to literal text.
   if (type == "") {
-    SG_LOG(SG_INPUT, SG_INFO, "No type provided for text chunk " << name
-          << " assuming \"literal\"");
+    SG_LOG( SG_COCKPIT, SG_INFO, "No type provided for text chunk " << name
+            << " assuming \"literal\"");
     type = "literal";
   }
 
@@ -462,27 +404,28 @@ readTextChunk (const SGPropertyNode * node)
 
                                // The value of a string property.
   else if (type == "text-value") {
-    SGValue * value =
-      fgGetValue(node->getStringValue("property"), true);
-    chunk = new FGTextLayer::Chunk(FGTextLayer::TEXT_VALUE, value, format);
+    SGPropertyNode * target =
+      fgGetNode(node->getStringValue("property"), true);
+    chunk = new FGTextLayer::Chunk(FGTextLayer::TEXT_VALUE, target, format);
   }
 
                                // The value of a float property.
   else if (type == "number-value") {
     string propName = node->getStringValue("property");
     float scale = node->getFloatValue("scale", 1.0);
-    SGValue * value = fgGetValue(propName, true);
-    chunk = new FGTextLayer::Chunk(FGTextLayer::DOUBLE_VALUE, value,
+    SGPropertyNode * target = fgGetNode(propName.c_str(), true);
+    chunk = new FGTextLayer::Chunk(FGTextLayer::DOUBLE_VALUE, target,
                                   format, scale);
   }
 
                                // Unknown type.
   else {
-    SG_LOG(SG_INPUT, SG_ALERT, "Unrecognized type " << type
-          << " for text chunk " << name);
+    SG_LOG( SG_COCKPIT, SG_ALERT, "Unrecognized type " << type
+            << " for text chunk " << name );
     return 0;
   }
 
+  readConditions(chunk, node);
   return chunk;
 }
 
@@ -523,9 +466,9 @@ readLayer (const SGPropertyNode * node, float w_scale, float h_scale)
 
 
   if (type == "") {
-    SG_LOG(SG_INPUT, SG_ALERT,
-          "No type supplied for layer " << name
-          << " assuming \"texture\"");
+    SG_LOG( SG_COCKPIT, SG_ALERT,
+            "No type supplied for layer " << name
+            << " assuming \"texture\"" );
     type = "texture";
   }
 
@@ -536,6 +479,19 @@ readLayer (const SGPropertyNode * node, float w_scale, float h_scale)
     layer = new FGTexturedLayer(texture, w, h);
   }
 
+                               // A group of sublayers.
+  else if (type == "group") {
+    layer = new FGGroupLayer();
+    for (int i = 0; i < node->nChildren(); i++) {
+      const SGPropertyNode * child = node->getChild(i);
+      cerr << "Trying child " << child->getName() << endl;
+      if (string(child->getName()) == "layer") {
+       cerr << "succeeded!" << endl;
+       ((FGGroupLayer *)layer)->addLayer(readLayer(child, w_scale, h_scale));
+      }
+    }
+  }
+
 
                                // A textual instrument layer.
   else if (type == "text") {
@@ -552,20 +508,21 @@ readLayer (const SGPropertyNode * node, float w_scale, float h_scale)
     tlayer->setPointSize(pointSize);
 
                                // Set the font.
-    // TODO
+    string fontName = node->getStringValue("font", "default");
+    tlayer->setFontName(fontName);
 
     const SGPropertyNode * chunk_group = node->getNode("chunks");
     if (chunk_group != 0) {
       int nChunks = chunk_group->nChildren();
       for (int i = 0; i < nChunks; i++) {
        const SGPropertyNode * node = chunk_group->getChild(i);
-       if (node->getName() == "chunk") {
+       if (string(node->getName()) == "chunk") {
          FGTextLayer::Chunk * chunk = readTextChunk(node);
          if (chunk != 0)
            tlayer->addChunk(chunk);
        } else {
-         SG_LOG(SG_INPUT, SG_INFO, "Skipping " << node->getName()
-                << " in chunks");
+         SG_LOG( SG_COCKPIT, SG_INFO, "Skipping " << node->getName()
+                  << " in chunks" );
        }
       }
       layer = tlayer;
@@ -574,13 +531,13 @@ readLayer (const SGPropertyNode * node, float w_scale, float h_scale)
 
                                // A switch instrument layer.
   else if (type == "switch") {
-    SGValue * value =
-      fgGetValue(node->getStringValue("property"), true);
+    SGPropertyNode * target =
+      fgGetNode(node->getStringValue("property"), true);
     FGInstrumentLayer * layer1 =
-      readLayer(node->getNode("layer1"), w_scale, h_scale);
+      readLayer(node->getNode("layer[0]"), w_scale, h_scale);
     FGInstrumentLayer * layer2 =
-      readLayer(node->getNode("layer2"), w_scale, h_scale);
-    layer = new FGSwitchLayer(w, h, value, layer1, layer2);
+      readLayer(node->getNode("layer[1]"), w_scale, h_scale);
+    layer = new FGSwitchLayer(w, h, target, layer1, layer2);
   }
 
                                // A built-in instrument layer.
@@ -592,21 +549,21 @@ readLayer (const SGPropertyNode * node, float w_scale, float h_scale)
     }
 
     else if (layerclass == "") {
-      SG_LOG(SG_INPUT, SG_ALERT, "No class provided for built-in layer "
-            << name);
+      SG_LOG( SG_COCKPIT, SG_ALERT, "No class provided for built-in layer "
+              << name );
       return 0;
     }
 
     else {
-      SG_LOG(SG_INPUT, SG_ALERT, "Unknown built-in layer class "
-            << layerclass);
+      SG_LOG( SG_COCKPIT, SG_ALERT, "Unknown built-in layer class "
+              << layerclass);
       return 0;
     }
   }
 
                                // An unknown type.
   else {
-    SG_LOG(SG_INPUT, SG_ALERT, "Unrecognized layer type " << type);
+    SG_LOG( SG_COCKPIT, SG_ALERT, "Unrecognized layer type " << type );
     delete layer;
     return 0;
   }
@@ -619,18 +576,19 @@ readLayer (const SGPropertyNode * node, float w_scale, float h_scale)
     int nTransformations = trans_group->nChildren();
     for (int i = 0; i < nTransformations; i++) {
       const SGPropertyNode * node = trans_group->getChild(i);
-      if (node->getName() == "transformation") {
+      if (string(node->getName()) == "transformation") {
        FGPanelTransformation * t = readTransformation(node, w_scale, h_scale);
        if (t != 0)
          layer->addTransformation(t);
       } else {
-       SG_LOG(SG_INPUT, SG_INFO, "Skipping " << node->getName()
-              << " in transformations");
+       SG_LOG( SG_COCKPIT, SG_INFO, "Skipping " << node->getName()
+                << " in transformations" );
       }
     }
   }
-  
-  SG_LOG(SG_INPUT, SG_INFO, "Read layer " << name);
+
+  readConditions(layer, node);
+  SG_LOG( SG_COCKPIT, SG_DEBUG, "Read layer " << name );
   return layer;
 }
 
@@ -658,8 +616,8 @@ readInstrument (const SGPropertyNode * node)
   int h = node->getIntValue("h-base", -1);
 
   if (x == -1 || y == -1) {
-    SG_LOG(SG_INPUT, SG_ALERT,
-          "x and y positions must be specified and >0");
+    SG_LOG( SG_COCKPIT, SG_ALERT,
+            "x and y positions must be specified and > 0" );
     return 0;
   }
 
@@ -674,7 +632,7 @@ readInstrument (const SGPropertyNode * node)
     h = real_h;
   }
 
-  SG_LOG(SG_INPUT, SG_INFO, "Reading instrument " << name);
+  SG_LOG( SG_COCKPIT, SG_DEBUG, "Reading instrument " << name );
 
   FGLayeredInstrument * instrument =
     new FGLayeredInstrument(x, y, w, h);
@@ -687,13 +645,13 @@ readInstrument (const SGPropertyNode * node)
     int nActions = action_group->nChildren();
     for (int i = 0; i < nActions; i++) {
       const SGPropertyNode * node = action_group->getChild(i);
-      if (node->getName() == "action") {
+      if (string(node->getName()) == "action") {
        FGPanelAction * action = readAction(node, w_scale, h_scale);
        if (action != 0)
          instrument->addAction(action);
       } else {
-       SG_LOG(SG_INPUT, SG_INFO, "Skipping " << node->getName()
-              << " in actions");
+       SG_LOG( SG_COCKPIT, SG_INFO, "Skipping " << node->getName()
+                << " in actions" );
       }
     }
   }
@@ -706,18 +664,19 @@ readInstrument (const SGPropertyNode * node)
     int nLayers = layer_group->nChildren();
     for (int i = 0; i < nLayers; i++) {
       const SGPropertyNode * node = layer_group->getChild(i);
-      if (node->getName() == "layer") {
+      if (string(node->getName()) == "layer") {
        FGInstrumentLayer * layer = readLayer(node, w_scale, h_scale);
        if (layer != 0)
          instrument->addLayer(layer);
       } else {
-       SG_LOG(SG_INPUT, SG_INFO, "Skipping " << node->getName()
-              << " in layers");
+       SG_LOG( SG_COCKPIT, SG_INFO, "Skipping " << node->getName()
+                << " in layers" );
       }
     }
   }
-    
-  SG_LOG(SG_INPUT, SG_INFO, "Done reading instrument " << name);
+
+  readConditions(instrument, node);
+  SG_LOG( SG_COCKPIT, SG_DEBUG, "Done reading instrument " << name );
   return instrument;
 }
 
@@ -728,10 +687,10 @@ readInstrument (const SGPropertyNode * node)
 FGPanel *
 readPanel (const SGPropertyNode * root)
 {
-  SG_LOG(SG_INPUT, SG_INFO, "Reading properties for panel " <<
-        root->getStringValue("name", "[Unnamed Panel]"));
+  SG_LOG( SG_COCKPIT, SG_INFO, "Reading properties for panel " <<
+          root->getStringValue("name", "[Unnamed Panel]") );
 
-  FGPanel * panel = new FGPanel(0, 0, 1024, 768);
+  FGPanel * panel = new FGPanel();
   panel->setWidth(root->getIntValue("w", 1024));
   panel->setHeight(root->getIntValue("h", 443));
 
@@ -744,10 +703,12 @@ readPanel (const SGPropertyNode * root)
   //
   // Grab the panel's initial offsets, default to 0, 0.
   //
-  if (!fgHasValue("/sim/panel/x-offset"))
+  if (!fgHasNode("/sim/panel/x-offset"))
     fgSetInt("/sim/panel/x-offset", root->getIntValue("x-offset", 0));
 
-  if (!fgHasValue("/sim/panel/y-offset"))
+  // conditional removed by jim wilson to allow panel xml code 
+  // with y-offset defined to work...
+  if (!fgHasNode("/sim/panel/y-offset"))
     fgSetInt("/sim/panel/y-offset", root->getIntValue("y-offset", 0));
 
   //
@@ -757,29 +718,82 @@ readPanel (const SGPropertyNode * root)
   if (bgTexture == "")
     bgTexture = "FOO";
   panel->setBackground(FGTextureManager::createTexture(bgTexture.c_str()));
-  SG_LOG(SG_INPUT, SG_INFO, "Set background texture to " << bgTexture);
+  SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << bgTexture );
+
+  //
+  // Get multibackground if any...
+  //
+  string mbgTexture = root->getStringValue("multibackground[0]");
+  if (!mbgTexture.empty()) {
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 0);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[1]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 1);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[2]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 2);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[3]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 3);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[4]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 4);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[5]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 5);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[6]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 6);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+    mbgTexture = root->getStringValue("multibackground[7]");
+    if (mbgTexture == "")
+      mbgTexture = "FOO";
+    panel->setMultiBackground(FGTextureManager::createTexture(mbgTexture.c_str()), 7);
+    SG_LOG( SG_COCKPIT, SG_INFO, "Set background texture to " << mbgTexture );
+
+  }
+  
 
 
   //
   // Create each instrument.
   //
-  SG_LOG(SG_INPUT, SG_INFO, "Reading panel instruments");
+  SG_LOG( SG_COCKPIT, SG_INFO, "Reading panel instruments" );
   const SGPropertyNode * instrument_group = root->getChild("instruments");
   if (instrument_group != 0) {
     int nInstruments = instrument_group->nChildren();
     for (int i = 0; i < nInstruments; i++) {
       const SGPropertyNode * node = instrument_group->getChild(i);
-      if (node->getName() == "instrument") {
+      if (string(node->getName()) == "instrument") {
        FGPanelInstrument * instrument = readInstrument(node);
        if (instrument != 0)
          panel->addInstrument(instrument);
       } else {
-       SG_LOG(SG_INPUT, SG_INFO, "Skipping " << node->getName()
-              << " in instruments section");
+       SG_LOG( SG_COCKPIT, SG_INFO, "Skipping " << node->getName()
+                << " in instruments section" );
       }
     }
   }
-  SG_LOG(SG_INPUT, SG_INFO, "Done reading panel instruments");
+  SG_LOG( SG_COCKPIT, SG_INFO, "Done reading panel instruments" );
 
 
   //
@@ -804,8 +818,10 @@ fgReadPanel (istream &input)
 {
   SGPropertyNode root;
 
-  if (!readProperties(input, &root)) {
-    SG_LOG(SG_INPUT, SG_ALERT, "Malformed property list for panel.");
+  try {
+    readProperties(input, &root);
+  } catch (const sg_exception &e) {
+    guiErrorMessage("Error reading panel: ", e);
     return 0;
   }
   return readPanel(&root);
@@ -825,8 +841,10 @@ fgReadPanel (const string &relative_path)
   path.append(relative_path);
   SGPropertyNode root;
 
-  if (!readProperties(path.str(), &root)) {
-    SG_LOG(SG_INPUT, SG_ALERT, "Malformed property list for panel.");
+  try {
+    readProperties(path.str(), &root);
+  } catch (const sg_exception &e) {
+    guiErrorMessage("Error reading panel: ", e);
     return 0;
   }
   return readPanel(&root);
@@ -835,3 +853,6 @@ fgReadPanel (const string &relative_path)
 
 
 // end of panel_io.cxx
+
+
+