uci: optimize update section in uci_set
authorJan Venekamp <jan@venekamp.net>
Sun, 20 Nov 2022 01:08:27 +0000 (02:08 +0100)
committerHauke Mehrtens <hauke@hauke-m.de>
Sat, 4 Mar 2023 18:39:32 +0000 (19:39 +0100)
Optimize for the case when there is no need to update the section and
the case there is no need to reallocate memory when updating a section
in uci_set.

Signed-off-by: Jan Venekamp <jan@venekamp.net>
list.c
tests/shunit2/tests.d/090_cli_options

diff --git a/list.c b/list.c
index 3e8a87c7d7f75febe05828cc563a8ff6a4ef5f1f..16402130ebf2d87751c3ab7466155e9e4bcebc60 100644 (file)
--- a/list.c
+++ b/list.c
@@ -727,14 +727,21 @@ int uci_set(struct uci_context *ctx, struct uci_ptr *ptr)
                        ptr->last = &ptr->o->e;
                }
        } else if (ptr->s && ptr->section) { /* update section */
-               struct uci_section *old = ptr->s;
-               ptr->s = uci_alloc_section(ptr->p, ptr->value, old->e.name, &old->e.list);
-               uci_section_transfer_options(ptr->s, old);
-               if (ptr->section == old->e.name)
-                       ptr->section = ptr->s->e.name;
-               uci_free_section(old);
-               ptr->s->package->n_section--;
-               ptr->last = &ptr->s->e;
+               if (!strcmp(ptr->s->type, ptr->value))
+                       return 0;
+
+               if (strlen(ptr->s->type) == strlen(ptr->value)) {
+                       strcpy(ptr->s->type, ptr->value);
+               } else {
+                       struct uci_section *old = ptr->s;
+                       ptr->s = uci_alloc_section(ptr->p, ptr->value, old->e.name, &old->e.list);
+                       uci_section_transfer_options(ptr->s, old);
+                       if (ptr->section == old->e.name)
+                               ptr->section = ptr->s->e.name;
+                       uci_free_section(old);
+                       ptr->s->package->n_section--;
+                       ptr->last = &ptr->s->e;
+               }
        } else {
                UCI_THROW(ctx, UCI_ERR_INVAL);
        }
index 55920a2af85856070f3bfa30d01d749182b76905..aff8316302f4249bfddc18dad40a7f4c3a36fc33 100644 (file)
@@ -11,8 +11,9 @@ test_add_delta() {
        # save new changes in "$new_savedir"
        mkdir -p "$new_savedir"
        touch "$new_savedir/delta"
-       $UCI -P "$new_savedir" set delta.sec0=sectype
+       $UCI -P "$new_savedir" set delta.sec0=tmptype
        $UCI -P "$new_savedir" add_list delta.sec0.li0=1
+       $UCI -P "$new_savedir" set delta.sec0=sectype
 
        assertEquals "delta.sec0='sectype'
 delta.sec0.li0+='0'" "$($UCI changes)"
@@ -22,8 +23,9 @@ delta.sec0.li0+='0'" "$($UCI changes)"
        assertTrue "$?"
        assertEquals "delta.sec0='sectype'
 delta.sec0.li0+='0'
-delta.sec0='sectype'
-delta.sec0.li0+='1'" "$cmdoutput"
+delta.sec0='tmptype'
+delta.sec0.li0+='1'
+delta.sec0='sectype'" "$cmdoutput"
 
        # check combined export.  Order matters here.
        cmdoutput="$($UCI -P "$new_savedir" export)"