remove uci_parse_tuple, preparation for adding a new api for lookups

diff --git a/Makefile b/Makefile
index ea7d5c7b6560dc4bb4449ca4528725c5fcb8504a..d16a987d058543db31205730cc066e3612d4a683 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,4 +1,4 @@
-VERSION=0.5
+VERSION=0.6
 
 # optional features
 PLUGIN_SUPPORT=1

diff --git a/history.c b/history.c
index 2b6dbef64718ea319b318c577c266cdf1425952e..4e975245de79caa2713cb315189c3d1bf442b307 100644 (file)
--- a/history.c
+++ b/history.c
@@ -88,7 +88,7 @@ int uci_add_history_path(struct uci_context *ctx, const char *dir)
        return 0;
 }
 
-static inline void uci_parse_history_tuple(struct uci_context *ctx, char **buf, char **package, char **section, char **option, char **value, int *cmd)
+static inline int uci_parse_history_tuple(struct uci_context *ctx, char **buf, struct uci_ptr *ptr)
 {
        int c = UCI_CMD_CHANGE;
 
@@ -111,53 +111,57 @@ static inline void uci_parse_history_tuple(struct uci_context *ctx, char **buf,
        if (c != UCI_CMD_CHANGE)
                *buf += 1;
 
-       if (cmd)
-               *cmd = c;
+       UCI_INTERNAL(uci_parse_ptr, ctx, ptr, *buf);
 
-       UCI_INTERNAL(uci_parse_tuple, ctx, *buf, package, section, option, value);
-       if (!*section[0])
-               UCI_THROW(ctx, UCI_ERR_PARSE);
+       if (!ptr->section)
+               goto error;
+       if (ptr->flags & UCI_LOOKUP_EXTENDED)
+               goto error;
+
+       switch(c) {
+       case UCI_CMD_RENAME:
+               if (!ptr->value || !uci_validate_name(ptr->value))
+                       goto error;
+               break;
+       case UCI_CMD_LIST_ADD:
+               if (!ptr->option)
+                       goto error;
+       }
 
+       return c;
+
+error:
+       UCI_THROW(ctx, UCI_ERR_INVAL);
+       return 0;
 }
 
 static void uci_parse_history_line(struct uci_context *ctx, struct uci_package *p, char *buf)
 {
        struct uci_element *e = NULL;
-       bool delete = false;
-       bool rename = false;
-       char *package = NULL;
-       char *section = NULL;
-       char *option = NULL;
-       char *value = NULL;
+       struct uci_ptr ptr;
        int cmd;
 
-       uci_parse_history_tuple(ctx, &buf, &package, &section, &option, &value, &cmd);
-       if (!package || (strcmp(package, p->e.name) != 0))
-               goto error;
-       if (!uci_validate_name(section))
-               goto error;
-       if (option && !uci_validate_name(option))
-               goto error;
-       if (rename && !uci_validate_str(value, (option || delete)))
+       cmd = uci_parse_history_tuple(ctx, &buf, &ptr);
+       if (strcmp(ptr.package, p->e.name) != 0)
                goto error;
 
        if (ctx->flags & UCI_FLAG_SAVED_HISTORY)
-               uci_add_history(ctx, &p->saved_history, cmd, section, option, value);
+               uci_add_history(ctx, &p->saved_history, cmd, ptr.section, ptr.option, ptr.value);
 
        switch(cmd) {
        case UCI_CMD_RENAME:
-               UCI_INTERNAL(uci_rename, ctx, p, section, option, value);
+               UCI_INTERNAL(uci_rename, ctx, p, ptr.section, ptr.option, ptr.value);
                break;
        case UCI_CMD_REMOVE:
-               UCI_INTERNAL(uci_delete, ctx, p, section, option);
+               UCI_INTERNAL(uci_delete, ctx, p, ptr.section, ptr.option);
                break;
        case UCI_CMD_LIST_ADD:
-               UCI_INTERNAL(uci_add_list, ctx, p, section, option, value, NULL);
+               UCI_INTERNAL(uci_add_list, ctx, p, ptr.section, ptr.option, ptr.value, NULL);
                break;
        case UCI_CMD_ADD:
        case UCI_CMD_CHANGE:
-               UCI_INTERNAL(uci_set, ctx, p, section, option, value, &e);
-               if (!option && e && (cmd == UCI_CMD_ADD))
+               UCI_INTERNAL(uci_set, ctx, p, ptr.section, ptr.option, ptr.value, &e);
+               if (!ptr.option && e && (cmd == UCI_CMD_ADD))
                        uci_to_section(e)->anonymous = true;
                break;
        }
@@ -260,10 +264,7 @@ static void uci_filter_history(struct uci_context *ctx, const char *name, const
        struct uci_element *e, *tmp;
        struct uci_list list;
        char *filename = NULL;
-       char *p = NULL;
-       char *s = NULL;
-       char *o = NULL;
-       char *v = NULL;
+       struct uci_ptr ptr;
        FILE *f = NULL;
 
        uci_list_init(&list);
@@ -291,13 +292,13 @@ static void uci_filter_history(struct uci_context *ctx, const char *name, const
                e = uci_alloc_generic(ctx, UCI_TYPE_HISTORY, pctx->buf, sizeof(struct uci_element));
                uci_list_add(&list, &e->list);
 
-               uci_parse_history_tuple(ctx, &buf, &p, &s, &o, &v, NULL);
+               uci_parse_history_tuple(ctx, &buf, &ptr);
                if (section) {
-                       if (!s || (strcmp(section, s) != 0))
+                       if (!ptr.section || (strcmp(section, ptr.section) != 0))
                                continue;
                }
                if (option) {
-                       if (!o || (strcmp(option, o) != 0))
+                       if (!ptr.option || (strcmp(option, ptr.option) != 0))
                                continue;
                }
                /* match, drop this element again */

diff --git a/list.c b/list.c
index 2331c7ef7946f0ac0bec7ba489e44e78a7264084..d7b9f69d1d3989877c78f64de25cb8e2843be5c8 100644 (file)
--- a/list.c
+++ b/list.c
@@ -264,27 +264,25 @@ static struct uci_element *uci_lookup_list(struct uci_list *list, const char *na
        return NULL;
 }
 
-int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *ptr)
+int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *str)
 {
        struct uci_package *p = NULL;
        struct uci_element *e;
        struct uci_section *s;
-       char *package = NULL;
-       char *section = NULL;
-       char *option = NULL;
+       struct uci_ptr ptr;
        char *idxstr, *t;
        int idx, c;
 
        UCI_HANDLE_ERR(ctx);
        UCI_ASSERT(ctx, res != NULL);
-       UCI_ASSERT(ctx, ptr != NULL);
+       UCI_ASSERT(ctx, str != NULL);
 
-       UCI_INTERNAL(uci_parse_tuple, ctx, ptr, &package, &section, &option, NULL);
+       UCI_INTERNAL(uci_parse_ptr, ctx, &ptr, str);
 
        /* look up the package first */
-       e = uci_lookup_list(&ctx->root, package);
+       e = uci_lookup_list(&ctx->root, ptr.package);
        if (!e) {
-               UCI_INTERNAL(uci_load, ctx, package, &p);
+               UCI_INTERNAL(uci_load, ctx, ptr.package, &p);
                if (!p)
                        goto notfound;
                e = &p->e;
@@ -292,25 +290,25 @@ int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *ptr)
                p = uci_to_package(e);
        }
 
-       if (!section)
+       if (!ptr.section)
                goto done;
 
        /* if the section name validates as a regular name, pass through
         * to the regular uci_lookup function call */
-       if (!*section || uci_validate_name(section)) {
-               UCI_INTERNAL(uci_lookup, ctx, &e, p, section, option);
+       if (!(ptr.flags & UCI_LOOKUP_EXTENDED)) {
+               UCI_INTERNAL(uci_lookup, ctx, &e, p, ptr.section, ptr.option);
                goto done;
        }
 
        /* name did not validate, that means we have an extended lookup call
         * parse it here. for now only the section index syntax is supported */
-       if (section[0] != '@')
+       if (ptr.section[0] != '@')
                goto error;
 
-       section++;
+       ptr.section++;
 
        /* parse the section index part */
-       idxstr = strchr(section, '[');
+       idxstr = strchr(ptr.section, '[');
        if (!idxstr)
                goto error;
        *idxstr = 0;
@@ -328,9 +326,9 @@ int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *ptr)
        if (t && *t)
                goto error;
 
-       if (!*section)
-               section = NULL;
-       if (section && !uci_validate_str(section, false))
+       if (!*ptr.section)
+               ptr.section = NULL;
+       if (ptr.section && !uci_validate_str(ptr.section, false))
                goto error;
 
        /* if the given index is negative, it specifies the section number from 
@@ -339,7 +337,7 @@ int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *ptr)
                c = 0;
                uci_foreach_element(&p->sections, e) {
                        s = uci_to_section(e);
-                       if (section && (strcmp(s->type, section) != 0))
+                       if (ptr.section && (strcmp(s->type, ptr.section) != 0))
                                continue;
 
                        c++;
@@ -350,7 +348,7 @@ int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *ptr)
        c = 0;
        uci_foreach_element(&p->sections, e) {
                s = uci_to_section(e);
-               if (section && (strcmp(s->type, section) != 0))
+               if (ptr.section && (strcmp(s->type, ptr.section) != 0))
                        continue;
 
                if (idx == c)
@@ -360,8 +358,8 @@ int uci_lookup_ext(struct uci_context *ctx, struct uci_element **res, char *ptr)
        goto notfound;
 
 found:
-       if (option)
-               e = uci_lookup_list(&s->options, option);
+       if (ptr.option)
+               e = uci_lookup_list(&s->options, ptr.option);
 done:
        *res = e;
        return 0;

diff --git a/uci.h b/uci.h
index 198ae2a5888a3da88cf69e72b19eb1a54df5cc77..fe05bb3cc34138bd531e22461d1011bd1ceea110 100644 (file)
--- a/uci.h
+++ b/uci.h
@@ -55,6 +55,7 @@ struct uci_list
        struct uci_list *prev;
 };
 
+struct uci_ptr;
 struct uci_element;
 struct uci_package;
 struct uci_section;
@@ -65,19 +66,6 @@ struct uci_backend;
 struct uci_parse_context;
 
 
-/**
- * uci_parse_tuple: Parse an uci tuple
- * @ctx: uci context
- * @str: input string
- * @package: output package pointer
- * @section: output section pointer
- * @option: output option pointer
- * @value: output value pointer
- *
- * format: <package>[.<section>[.<option>]][=<value>]
- */
-extern int uci_parse_tuple(struct uci_context *ctx, char *str, char **package, char **section, char **option, char **value);
-
 /**
  * uci_alloc_context: Allocate a new uci context
  */
@@ -148,6 +136,15 @@ extern int uci_unload(struct uci_context *ctx, struct uci_package *p);
  */
 extern int uci_lookup(struct uci_context *ctx, struct uci_element **res, struct uci_package *package, const char *section, const char *option);
 
+/**
+ * uci_lookup_ptr: Split an uci tuple string and look up elements
+ * @ctx: uci context
+ * @ptr: lookup result struct
+ * @str: uci tuple string to look up
+ * @extended: allow extended syntax lookup
+ */
+extern int uci_lookup_ptr(struct uci_context *ctx, struct uci_ptr *ptr, char *str, bool extended);
+
 /**
  * uci_lookup_ext: Extended lookup for an uci element
  *
@@ -155,9 +152,8 @@ extern int uci_lookup(struct uci_context *ctx, struct uci_element **res, struct
  * @res: where to store the result
  * @ptr: uci pointer tuple
  *
- * Looks up an element using the extended syntax, which is a superset of what
- * uci_parse_tuple accepts. It can look up sections by an index with an optional
- * type.
+ * Looks up an element using the extended syntax.
+ * It can look up sections by an index with an optional type.
  *
  * Examples:
  *   network.@interface[0].ifname ('ifname' option of the first interface section)
@@ -333,13 +329,14 @@ extern bool uci_validate_text(const char *str);
 
 /* UCI data structures */
 enum uci_type {
-       UCI_TYPE_HISTORY = 0,
-       UCI_TYPE_PACKAGE = 1,
-       UCI_TYPE_SECTION = 2,
-       UCI_TYPE_OPTION = 3,
-       UCI_TYPE_PATH = 4,
-       UCI_TYPE_BACKEND = 5,
-       UCI_TYPE_ITEM = 6,
+       UCI_TYPE_UNSPEC = 0,
+       UCI_TYPE_HISTORY = 1,
+       UCI_TYPE_PACKAGE = 2,
+       UCI_TYPE_SECTION = 3,
+       UCI_TYPE_OPTION = 4,
+       UCI_TYPE_PATH = 5,
+       UCI_TYPE_BACKEND = 6,
+       UCI_TYPE_ITEM = 7,
 };
 
 enum uci_option_type {
@@ -455,6 +452,25 @@ struct uci_history
        char *value;
 };
 
+struct uci_ptr
+{
+       enum uci_type target;
+       enum {
+               UCI_LOOKUP_DONE =     (1 << 0),
+               UCI_LOOKUP_COMPLETE = (1 << 1),
+               UCI_LOOKUP_EXTENDED = (1 << 2),
+       } flags;
+
+       struct uci_package *p;
+       struct uci_section *s;
+       struct uci_option *o;
+
+       char *package;
+       char *section;
+       char *option;
+       char *value;
+};
+
 
 /* linked list handling */
 #ifndef offsetof

diff --git a/util.c b/util.c
index dde7bfc96dd15c907bc387eac6faa02ab5d1b4b3..d5bcd8a7b99a05a1c9a16e575e37dfca6fcc4c0a 100644 (file)
--- a/util.c
+++ b/util.c
@@ -117,59 +117,57 @@ static void uci_alloc_parse_context(struct uci_context *ctx)
        ctx->pctx = (struct uci_parse_context *) uci_malloc(ctx, sizeof(struct uci_parse_context));
 }
 
-int uci_parse_tuple(struct uci_context *ctx, char *str, char **package, char **section, char **option, char **value)
+int uci_parse_ptr(struct uci_context *ctx, struct uci_ptr *ptr, char *str)
 {
        char *last = NULL;
-       bool internal = ctx->internal;
 
        UCI_HANDLE_ERR(ctx);
-       UCI_ASSERT(ctx, str && package && section && option);
+       UCI_ASSERT(ctx, str);
+       UCI_ASSERT(ctx, ptr);
 
+       memset(ptr, 0, sizeof(struct uci_ptr));
+
+       /* value */
        last = strchr(str, '=');
        if (last) {
                *last = 0;
                last++;
+               ptr->value = last;
        }
 
-       *package = strsep(&str, ".");
-       if (!*package || !uci_validate_str(*package, false))
+       ptr->package = strsep(&str, ".");
+       if (!ptr->package)
                goto error;
 
-       *section = strsep(&str, ".");
-       *option = NULL;
-       if (value)
-               *value = NULL;
-       if (!*section)
+       ptr->section = strsep(&str, ".");
+       if (!ptr->section) {
+               ptr->target = UCI_TYPE_PACKAGE;
                goto lastval;
+       }
 
-       *option = strsep(&str, ".");
-       if (!*option)
+       ptr->option = strsep(&str, ".");
+       if (!ptr->option) {
+               ptr->target = UCI_TYPE_SECTION;
                goto lastval;
-
-lastval:
-       if (last) {
-               if (!value)
-                       goto error;
-
-               if (!*last)
-                       goto error;
-               *value = last;
+       } else {
+               ptr->target = UCI_TYPE_OPTION;
        }
 
-       if (*section && *section[0] && !internal && !uci_validate_name(*section))
+lastval:
+       if (ptr->package && !uci_validate_str(ptr->package, false))
                goto error;
-       if (*option && !uci_validate_name(*option))
+       if (ptr->section && !uci_validate_name(ptr->section))
+               ptr->flags |= UCI_LOOKUP_EXTENDED;
+       if (ptr->option && !uci_validate_name(ptr->option))
                goto error;
-       if (value && *value && !uci_validate_text(*value))
+       if (ptr->value && !uci_validate_text(ptr->value))
                goto error;
 
-       goto done;
+       return 0;
 
 error:
+       memset(ptr, 0, sizeof(struct uci_ptr));
        UCI_THROW(ctx, UCI_ERR_PARSE);
-
-done:
-       return 0;
 }