Don't strip leading whitespace from menu labels (Fix bug 4782)

[dana/openbox.git] / obt / xml.c

diff --git a/obt/xml.c b/obt/xml.c
index fde3b76..5b7e77b 100644 (file)
--- a/obt/xml.c
+++ b/obt/xml.c
@@ -321,16 +321,22 @@ void obt_xml_tree_from_root(ObtXmlInst *i)
     obt_xml_tree(i, i->root->children);
 }
 
     obt_xml_tree(i, i->root->children);
 }
 

-gchar *obt_xml_node_string(xmlNodePtr node)
+gchar *obt_xml_node_string_unstripped(xmlNodePtr node)

 {
     xmlChar *c = xmlNodeGetContent(node);
     gchar *s;
 {
     xmlChar *c = xmlNodeGetContent(node);
     gchar *s;

-    if (c) g_strstrip((char*)c); /* strip leading/trailing whitespace */

     s = g_strdup(c ? (gchar*)c : "");
     xmlFree(c);
     return s;
 }
 
     s = g_strdup(c ? (gchar*)c : "");
     xmlFree(c);
     return s;
 }
 

+gchar *obt_xml_node_string(xmlNodePtr node)
+{
+    gchar* result = obt_xml_node_string_unstripped(node);
+    g_strstrip(result); /* strip leading/trailing whitespace */
+    return result;
+}
+

 gint obt_xml_node_int(xmlNodePtr node)
 {
     xmlChar *c = xmlNodeGetContent(node);
 gint obt_xml_node_int(xmlNodePtr node)
 {
     xmlChar *c = xmlNodeGetContent(node);


@@ -413,13 +419,12 @@ gboolean obt_xml_attr_int(xmlNodePtr node, const gchar *name, gint *value)
     return r;
 }
 
     return r;
 }
 

-gboolean obt_xml_attr_string(xmlNodePtr node, const gchar *name,
-                             gchar **value)
+gboolean obt_xml_attr_string_unstripped(xmlNodePtr node, const gchar *name,
+                                        gchar **value)

 {
     xmlChar *c = xmlGetProp(node, (const xmlChar*) name);
     gboolean r = FALSE;
     if (c) {
 {
     xmlChar *c = xmlGetProp(node, (const xmlChar*) name);
     gboolean r = FALSE;
     if (c) {

-        g_strstrip((char*)c); /* strip leading/trailing whitespace */

         *value = g_strdup((gchar*)c);
         r = TRUE;
     }
         *value = g_strdup((gchar*)c);
         r = TRUE;
     }


@@ -427,6 +432,15 @@ gboolean obt_xml_attr_string(xmlNodePtr node, const gchar *name,
     return r;
 }
 
     return r;
 }
 

+gboolean obt_xml_attr_string(xmlNodePtr node, const gchar *name,
+                             gchar **value)
+{
+    gboolean result = obt_xml_attr_string_unstripped(node, name, value);
+    if (result)
+        g_strstrip(*value); /* strip leading/trailing whitespace */
+    return result;
+}
+

 gboolean obt_xml_attr_contains(xmlNodePtr node, const gchar *name,
                                const gchar *val)
 {
 gboolean obt_xml_attr_contains(xmlNodePtr node, const gchar *name,
                                const gchar *val)
 {