Fixed regression introduced in 50107
authorSergey Sharybin <sergey.vfx@gmail.com>
Wed, 22 Aug 2012 17:22:04 +0000 (17:22 +0000)
committerSergey Sharybin <sergey.vfx@gmail.com>
Wed, 22 Aug 2012 17:22:04 +0000 (17:22 +0000)
Shader preview job localizes material without referencing it as a user,
so don't need to unreference itself as a user from this material.

Added BKE_material_free_ex function which could skip user dereferencing.
This also removed old hack with mtex users.

source/blender/blenkernel/BKE_material.h
source/blender/blenkernel/intern/material.c
source/blender/editors/render/render_preview.c

index 030af85ba849ddd1e18fe9c3a5b3d5749713fb1b..dd1b1a7752bffeb8e90180e0580d37fee5655460 100644 (file)
@@ -48,6 +48,7 @@ struct Scene;
 
 void init_def_material(void);
 void BKE_material_free(struct Material *sc); 
+void BKE_material_free_ex(struct Material *ma, int do_id_user);
 void test_object_materials(struct ID *id);
 void resize_object_material(struct Object *ob, const short totcol);
 void init_material(struct Material *ma);
index 8a26a1b52c5c9805af3c1ca2b461f18a776dd1be..fd167c69e628f538d2ac1be7848543a7779281d5 100644 (file)
@@ -79,13 +79,19 @@ void init_def_material(void)
 
 /* not material itself */
 void BKE_material_free(Material *ma)
+{
+       BKE_material_free_ex(ma, TRUE);
+}
+
+/* not material itself */
+void BKE_material_free_ex(Material *ma, int do_id_user)
 {
        MTex *mtex;
        int a;
        
        for (a = 0; a < MAX_MTEX; a++) {
                mtex = ma->mtex[a];
-               if (mtex && mtex->tex) mtex->tex->id.us--;
+               if (do_id_user && mtex && mtex->tex) mtex->tex->id.us--;
                if (mtex) MEM_freeN(mtex);
        }
        
@@ -101,7 +107,7 @@ void BKE_material_free(Material *ma)
        
        /* is no lib link block, but material extension */
        if (ma->nodetree) {
-               ntreeFreeTree(ma->nodetree);
+               ntreeFreeTree_ex(ma->nodetree, do_id_user);
                MEM_freeN(ma->nodetree);
        }
 
index 84319061d52951d4087742ed37f579535dcc54d8..add2cbd566bf1d7bfaeef7fc03e1f937cf173ca3 100644 (file)
@@ -749,7 +749,6 @@ static void shader_preview_free(void *customdata)
        
        if (sp->matcopy) {
                struct IDProperty *properties;
-               int a;
                
                /* node previews */
                shader_preview_updatejob(sp);
@@ -757,13 +756,7 @@ static void shader_preview_free(void *customdata)
                /* get rid of copied material */
                BLI_remlink(&pr_main->mat, sp->matcopy);
                
-               /* BKE_material_free decrements texture, prevent this. hack alert! */
-               for (a = 0; a < MAX_MTEX; a++) {
-                       MTex *mtex = sp->matcopy->mtex[a];
-                       if (mtex && mtex->tex) mtex->tex = NULL;
-               }
-               
-               BKE_material_free(sp->matcopy);
+               BKE_material_free_ex(sp->matcopy, FALSE);
 
                properties = IDP_GetProperties((ID *)sp->matcopy, FALSE);
                if (properties) {