ListBase: Add insert-replace function

[blender.git] / source / blender / blenkernel / intern / idprop.c

diff --git a/source/blender/blenkernel/intern/idprop.c b/source/blender/blenkernel/intern/idprop.c
index a596869e5e8d628df11117d620b38a16899d1c28..6ebdd371acbcdb54ed5c26a1878fb6b9941c3b2a 100644 (file)
--- a/source/blender/blenkernel/intern/idprop.c
+++ b/source/blender/blenkernel/intern/idprop.c
@@ -503,14 +503,9 @@ void IDP_SyncGroupValues(IDProperty *dest, const IDProperty *src)
                                        break;
                                default:
                                {
-                                       IDProperty *tmp = other;
-                                       IDProperty *copy = IDP_CopyProperty(prop);
-
-                                       BLI_insertlinkafter(&dest->data.group, other, copy);
-                                       BLI_remlink(&dest->data.group, tmp);
-
-                                       IDP_FreeProperty(tmp);
-                                       MEM_freeN(tmp);
+                                       BLI_insertlinkreplace(&dest->data.group, other, IDP_CopyProperty(prop));
+                                       IDP_FreeProperty(other);
+                                       MEM_freeN(other);
                                        break;
                                }
                        }
@@ -530,11 +525,9 @@ void IDP_SyncGroupTypes(IDProperty *dst, const IDProperty *src, const bool do_ar
                        if ((prop_dst->type != prop_src->type || prop_dst->subtype != prop_src->subtype) ||
                            (do_arraylen && ELEM(prop_dst->type, IDP_ARRAY, IDP_IDPARRAY) && (prop_src->len != prop_dst->len)))
                        {
-                               IDP_FreeFromGroup(dst, prop_dst);
-                               prop_dst = IDP_CopyProperty(prop_src);
-
-                               dst->len++;
-                               BLI_insertlinkbefore(&dst->data.group, prop_dst_next, prop_dst);
+                               BLI_insertlinkreplace(&dst->data.group, prop_dst, IDP_CopyProperty(prop_src));
+                               IDP_FreeProperty(prop_dst);
+                               MEM_freeN(prop_dst);
                        }
                        else if (prop_dst->type == IDP_GROUP) {
                                IDP_SyncGroupTypes(prop_dst, prop_src, do_arraylen);
@@ -559,11 +552,7 @@ void IDP_ReplaceGroupInGroup(IDProperty *dest, const IDProperty *src)
        for (prop = src->data.group.first; prop; prop = prop->next) {
                for (loop = dest->data.group.first; loop; loop = loop->next) {
                        if (STREQ(loop->name, prop->name)) {
-                               IDProperty *copy = IDP_CopyProperty(prop);
-
-                               BLI_insertlinkafter(&dest->data.group, loop, copy);
-
-                               BLI_remlink(&dest->data.group, loop);
+                               BLI_insertlinkreplace(&dest->data.group, loop, IDP_CopyProperty(prop));
                                IDP_FreeProperty(loop);
                                MEM_freeN(loop);
                                break;
@@ -590,9 +579,7 @@ void IDP_ReplaceInGroup_ex(IDProperty *group, IDProperty *prop, IDProperty *prop
        BLI_assert(prop_exist == IDP_GetPropertyFromGroup(group, prop->name));
 
        if ((prop_exist = IDP_GetPropertyFromGroup(group, prop->name))) {
-               BLI_insertlinkafter(&group->data.group, prop_exist, prop);
-               
-               BLI_remlink(&group->data.group, prop_exist);
+               BLI_insertlinkreplace(&group->data.group, prop_exist, prop);
                IDP_FreeProperty(prop_exist);
                MEM_freeN(prop_exist);
        }