]> Dogcows Code - chaz/openbox/blobdiff - obt/ddparse.c
improved .desktop parsing.
[chaz/openbox] / obt / ddparse.c
index fe7feb93cc3eea25440c8c4ccb416c060b561e1c..aaecda0150c95fa2ecce8cf783613925a59ba263 100644 (file)
@@ -17,6 +17,7 @@
 */
 
 #include "obt/ddparse.h"
+#include "obt/link.h"
 #ifdef HAVE_STRING_H
 #include <string.h>
 #endif
@@ -34,9 +35,19 @@ typedef gboolean (*ObtDDParseValueFunc)(gchar *key, const gchar *val,
                                         ObtDDParse *parse, gboolean *error);
 
 
+enum {
+    DE_TYPE             = 1 << 0,
+    DE_TYPE_APPLICATION = 1 << 1,
+    DE_TYPE_LINK        = 1 << 2,
+    DE_NAME             = 1 << 3,
+    DE_EXEC             = 1 << 4,
+    DE_URL              = 1 << 5
+};
+
 struct _ObtDDParse {
     gchar *filename;
     gulong lineno;
+    gulong flags;
     ObtDDParseGroup *group;
     /* the key is a group name, the value is a ObtDDParseGroup */
     GHashTable *group_hash;
@@ -68,6 +79,7 @@ static void parse_error(const gchar *m, const ObtDDParse *const parse,
 static void parse_value_free(ObtDDParseValue *v)
 {
     switch (v->type) {
+    case OBT_DDPARSE_EXEC:
     case OBT_DDPARSE_STRING:
     case OBT_DDPARSE_LOCALESTRING:
         g_free(v->value.string); break;
@@ -77,8 +89,9 @@ static void parse_value_free(ObtDDParseValue *v)
         v->value.strings.n = 0;
         break;
     case OBT_DDPARSE_BOOLEAN:
-        break;
     case OBT_DDPARSE_NUMERIC:
+    case OBT_DDPARSE_ENUM_TYPE:
+    case OBT_DDPARSE_ENVIRONMENTS:
         break;
     default:
         g_assert_not_reached();
@@ -144,7 +157,12 @@ static gchar* parse_value_string(const gchar *in,
     i = in; o = out;
     backslash = FALSE;
     while (i < end) {
-        const gchar *next = locale ? g_utf8_find_next_char(i, end) : i+1;
+        const gchar *next;
+
+        /* find the next character in the string */
+        if (!locale) next = i+1;
+        else if (!(next = g_utf8_find_next_char(i, end))) next = end;
+
         if (backslash) {
             switch(*i) {
             case 's': *o++ = ' '; break;
@@ -179,7 +197,69 @@ static gchar* parse_value_string(const gchar *in,
         i = next;
     }
     *o = '\0';
-    return o;
+    return out;
+}
+
+static guint parse_value_environments(const gchar *in,
+                                      const ObtDDParse *const parse,
+                                      gboolean *error)
+{
+    const gchar *s;
+    guint mask = 0;
+
+    s = in;
+    while (*s) {
+        switch (*(s++)) {
+        case 'G':
+            if (strcmp(s, "NOME") == 0) {
+                mask |= OBT_LINK_ENV_GNOME;
+                s += 4;
+            }
+            break;
+        case 'K':
+            if (strcmp(s, "DE") == 0) {
+                mask |= OBT_LINK_ENV_KDE;
+                s += 2;
+            }
+            break;
+        case 'L':
+            if (strcmp(s, "XDE") == 0) {
+                mask |= OBT_LINK_ENV_LXDE;
+                s += 3;
+            }
+            break;
+        case 'R':
+            if (strcmp(s, "OX") == 0) {
+                mask |= OBT_LINK_ENV_ROX;
+                s += 2;
+            }
+            break;
+        case 'X':
+            if (strcmp(s, "FCE") == 0) {
+                mask |= OBT_LINK_ENV_XFCE;
+                s += 3;
+            }
+            break;
+        case 'O':
+            switch (*(s++)) {
+            case 'l':
+                if (strcmp(s, "d") == 0) {
+                    mask |= OBT_LINK_ENV_OLD;
+                    s += 1;
+                }
+                break;
+            case 'P':
+                if (strcmp(s, "ENBOX") == 0) {
+                    mask |= OBT_LINK_ENV_OPENBOX;
+                    s += 5;
+                }
+                break;
+            }
+        }
+        /* find the next string, or the end of the sequence */
+        while (*s && *s != ';') ++s;
+    }
+    return mask;
 }
 
 static gboolean parse_value_boolean(const gchar *in,
@@ -348,9 +428,10 @@ static void parse_key_value(const gchar *buf, gulong len,
               ((guchar)buf[i] >= '0' && (guchar)buf[i] <= '9') ||
               ((guchar)buf[i] == '-'))) {
             /* not part of the key */
-            keyend = i;
             break;
         }
+    keyend = i;
+
     if (keyend < 1) {
         parse_error("Empty key", parse, error);
         return;
@@ -440,7 +521,7 @@ static gboolean parse_desktop_entry_value(gchar *key, const gchar *val,
         break;
     case 'E': /* Exec */
         if (strcmp(key+1, "xec")) return FALSE;
-        v.type = OBT_DDPARSE_STRING; break;
+        v.type = OBT_DDPARSE_EXEC; parse->flags |= DE_EXEC; break;
     case 'G': /* GenericName */
         if (strcmp(key+1, "enericName")) return FALSE;
         v.type = OBT_DDPARSE_LOCALESTRING; break;
@@ -457,7 +538,7 @@ static gboolean parse_desktop_entry_value(gchar *key, const gchar *val,
         switch (key[1]) {
         case 'a': /* Name */
             if (strcmp(key+2, "me")) return FALSE;
-            v.type = OBT_DDPARSE_LOCALESTRING; break;
+            v.type = OBT_DDPARSE_LOCALESTRING; parse->flags |= DE_NAME; break;
         case 'o':
             switch (key[2]) {
             case 'D': /* NoDisplay */
@@ -503,14 +584,14 @@ static gboolean parse_desktop_entry_value(gchar *key, const gchar *val,
             v.type = OBT_DDPARSE_STRING; break;
         case 'y': /* Type */
             if (strcmp(key+2, "pe")) return FALSE;
-            v.type = OBT_DDPARSE_STRING; break;
+            v.type = OBT_DDPARSE_ENUM_TYPE; parse->flags |= DE_TYPE; break;
         default:
             return FALSE;
         }
         break;
     case 'U': /* URL */
         if (strcmp(key+1, "RL")) return FALSE;
-        v.type = OBT_DDPARSE_STRING; break;
+        v.type = OBT_DDPARSE_STRING; parse->flags |= DE_URL; break;
     case 'V': /* MimeType */
         if (strcmp(key+1, "ersion")) return FALSE;
         v.type = OBT_DDPARSE_STRING; break;
@@ -520,6 +601,57 @@ static gboolean parse_desktop_entry_value(gchar *key, const gchar *val,
 
     /* parse the value */
     switch (v.type) {
+    case OBT_DDPARSE_EXEC: {
+        gchar *c, *m;
+        gboolean percent;
+        gboolean found;
+
+        v.value.string = parse_value_string(val, FALSE, NULL, parse, error);
+        g_assert(v.value.string);
+
+        /* an exec string can only contain one of the file/url-opening %'s */
+        percent = found = FALSE;
+        for (c = v.value.string; *c; ++c) {
+            if (*c == '%') percent = !percent;
+            if (percent) {
+                switch (*c) {
+                case 'f':
+                case 'F':
+                case 'u':
+                case 'U':
+                    if (found) {
+                        m = g_strdup_printf("Malformed Exec key, "
+                                            "extraneous %%%c", *c);
+                        parse_error(m, parse, error);
+                        g_free(m);
+                    }
+                    found = TRUE;
+                    break;
+                case 'd':
+                case 'D':
+                case 'n':
+                case 'N':
+                case 'v':
+                case 'm':
+                    m = g_strdup_printf("Malformed Exec key, "
+                                        "uses deprecated %%%c", *c);
+                    parse_error(m, parse, NULL); /* just a warning */
+                    g_free(m);
+                    break;
+                case 'i':
+                case 'c':
+                case 'k':
+                    break;
+                default:
+                    m = g_strdup_printf("Malformed Exec key, "
+                                        "uses unknown %%%c", *c);
+                    parse_error(m, parse, NULL); /* just a warning */
+                    g_free(m);
+                }
+            }
+        }
+        break;
+    }
     case OBT_DDPARSE_STRING:
         v.value.string = parse_value_string(val, FALSE, NULL, parse, error);
         g_assert(v.value.string);
@@ -546,6 +678,25 @@ static gboolean parse_desktop_entry_value(gchar *key, const gchar *val,
     case OBT_DDPARSE_NUMERIC:
         v.value.numeric = parse_value_numeric(val, parse, error);
         break;
+    case OBT_DDPARSE_ENUM_TYPE:
+        if (val[0] == 'A' && strcmp(val+1, "pplication") == 0) {
+            v.value.enumerable = OBT_LINK_TYPE_APPLICATION;
+            parse->flags |= DE_TYPE_APPLICATION;
+        }
+        else if (val[0] == 'L' && strcmp(val+1, "ink") == 0) {
+            v.value.enumerable = OBT_LINK_TYPE_URL;
+            parse->flags |= DE_TYPE_LINK;
+        }
+        else if (val[0] == 'D' && strcmp(val+1, "irectory") == 0)
+            v.value.enumerable = OBT_LINK_TYPE_DIRECTORY;
+        else {
+            parse_error("Unknown Type", parse, error);
+            return FALSE;
+        }
+        break;
+    case OBT_DDPARSE_ENVIRONMENTS:
+        v.value.environments = parse_value_environments(val, parse, error);
+        break;
     default:
         g_assert_not_reached();
     }
@@ -583,17 +734,40 @@ GHashTable* obt_ddparse_file(const gchar *name, GSList *paths)
         if ((f = fopen(path, "r"))) {
             parse.filename = path;
             parse.lineno = 1;
-            success = parse_file(f, &parse);
+            parse.flags = 0;
+            if ((success = parse_file(f, &parse))) {
+                /* check that required keys exist */
+
+                if (!(parse.flags & DE_TYPE)) {
+                    g_warning("Missing Type key in %s", path);
+                    success = FALSE;
+                }
+                if (!(parse.flags & DE_NAME)) {
+                    g_warning("Missing Name key in %s", path);
+                    success = FALSE;
+                }
+                if (parse.flags & DE_TYPE_APPLICATION &&
+                    !(parse.flags & DE_EXEC))
+                {
+                    g_warning("Missing Exec key for Application in %s",
+                              path);
+                    success = FALSE;
+                }
+                else if (parse.flags & DE_TYPE_LINK && !(parse.flags & DE_URL))
+                {
+                    g_warning("Missing URL key for Link in %s", path);
+                    success = FALSE;
+                }
+            }
             fclose(f);
         }
         g_free(path);
     }
     if (!success) {
         g_hash_table_destroy(parse.group_hash);
-        return NULL;
+        parse.group_hash = NULL;
     }
-    else
-        return parse.group_hash;
+    return parse.group_hash;
 }
 
 GHashTable* obt_ddparse_group_keys(ObtDDParseGroup *g)
This page took 0.026494 seconds and 4 git commands to generate.