Revision: 35997
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=35997
Author:   jhk
Date:     2011-04-04 13:47:28 +0000 (Mon, 04 Apr 2011)
Log Message:
-----------
Fix for [#26747] Crash accessing dynamic hair RNA
* Removed hairkey.co_dynamic. The hairkey access is now simplified so that 
hairkey.co are the dynamic coordinates whenever the hair is dynamic.

Modified Paths:
--------------
    trunk/blender/source/blender/makesrna/intern/rna_particle.c

Modified: trunk/blender/source/blender/makesrna/intern/rna_particle.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_particle.c 2011-04-04 
13:33:31 UTC (rev 35996)
+++ trunk/blender/source/blender/makesrna/intern/rna_particle.c 2011-04-04 
13:47:28 UTC (rev 35997)
@@ -183,58 +183,17 @@
        rna_ParticleHairKey_location_object_info(ptr, &psmd, &pa);
 
        if(pa) {
-               float hairmat[4][4];
-               psys_mat_hair_to_object(ob, psmd->dm, psmd->psys->part->from, 
pa, hairmat);
-               copy_v3_v3(values, hkey->co);
-               mul_m4_v3(hairmat, values);
-       }
-       else {
-               zero_v3(values);
-       }
-}
+               DerivedMesh *hairdm = (psmd->psys->flag & PSYS_HAIR_DYNAMICS) ? 
psmd->psys->hair_out_dm : NULL;
 
-static void rna_ParticleHairKey_location_object_set(PointerRNA *ptr, const 
float *values)
-{
-       HairKey *hkey= (HairKey *)ptr->data;
-       Object *ob = (Object *)ptr->id.data;
-       ParticleSystemModifierData *psmd;
-       ParticleData *pa;
-
-       rna_ParticleHairKey_location_object_info(ptr, &psmd, &pa);
-
-       if(pa) {
-               float hairmat[4][4];
-               float imat[4][4];
-
-               psys_mat_hair_to_object(ob, psmd->dm, psmd->psys->part->from, 
pa, hairmat);
-               invert_m4_m4(imat, hairmat);
-               copy_v3_v3(hkey->co, values);
-               mul_m4_v3(imat, hkey->co);
-       }
-       else {
-               zero_v3(hkey->co);
-       }
-}
-
-static void rna_ParticleHairKey_dynamic_location_object_get(PointerRNA *ptr, 
float *values)
-{
-       HairKey *hkey= (HairKey *)ptr->data;
-       // Object *ob = (Object *)ptr->id.data;
-       ParticleSystemModifierData *psmd;
-       ParticleData *pa;
-
-       rna_ParticleHairKey_location_object_info(ptr, &psmd, &pa);
-
-       if(pa) {
-               ParticleSystem *psys = psmd->psys;
-               DerivedMesh *hairdm = (psys->flag & PSYS_HAIR_DYNAMICS) ? 
psys->hair_out_dm : NULL;
-
                if(hairdm) {
                        MVert *mvert = CDDM_get_vert(hairdm, pa->hair_index + 
(hkey - pa->hair));
                        copy_v3_v3(values, mvert->co);
                }
                else {
-                       rna_ParticleHairKey_dynamic_location_object_get(ptr, 
values);
+                       float hairmat[4][4];
+                       psys_mat_hair_to_object(ob, psmd->dm, 
psmd->psys->part->from, pa, hairmat);
+                       copy_v3_v3(values, hkey->co);
+                       mul_m4_v3(hairmat, values);
                }
        }
        else {
@@ -242,25 +201,30 @@
        }
 }
 
-static void rna_ParticleHairKey_dynamic_location_object_set(PointerRNA *ptr, 
const float *values)
+static void rna_ParticleHairKey_location_object_set(PointerRNA *ptr, const 
float *values)
 {
        HairKey *hkey= (HairKey *)ptr->data;
-       // Object *ob = (Object *)ptr->id.data;
+       Object *ob = (Object *)ptr->id.data;
        ParticleSystemModifierData *psmd;
        ParticleData *pa;
 
        rna_ParticleHairKey_location_object_info(ptr, &psmd, &pa);
 
        if(pa) {
-               ParticleSystem *psys = psmd->psys;
-               DerivedMesh *hairdm = (psys->flag & PSYS_HAIR_DYNAMICS) ? 
psys->hair_out_dm : NULL;
+               DerivedMesh *hairdm = (psmd->psys->flag & PSYS_HAIR_DYNAMICS) ? 
psmd->psys->hair_out_dm : NULL;
 
                if(hairdm) {
                        MVert *mvert = CDDM_get_vert(hairdm, pa->hair_index + 
(hkey - pa->hair));
                        copy_v3_v3(mvert->co, values);
                }
                else {
-                       rna_ParticleHairKey_dynamic_location_object_set(ptr, 
values);
+                       float hairmat[4][4];
+                       float imat[4][4];
+
+                       psys_mat_hair_to_object(ob, psmd->dm, 
psmd->psys->part->from, pa, hairmat);
+                       invert_m4_m4(imat, hairmat);
+                       copy_v3_v3(hkey->co, values);
+                       mul_m4_v3(imat, hkey->co);
                }
        }
        else {
@@ -941,11 +905,6 @@
        prop= RNA_def_property(srna, "co_hair_space", PROP_FLOAT, 
PROP_TRANSLATION);
        RNA_def_property_float_sdna(prop, NULL, "co");
        RNA_def_property_ui_text(prop, "Location", "Location of the hair key in 
its internal coordinate system, relative to the emitting face");
-
-       prop= RNA_def_property(srna, "co_dynamic", PROP_FLOAT, 
PROP_TRANSLATION);
-       RNA_def_property_array(prop, 3);
-       RNA_def_property_ui_text(prop, "Location (Dynamic)", "Location of the 
hair key for the current frame with hair dynamics applied");
-       RNA_def_property_float_funcs(prop, 
"rna_ParticleHairKey_dynamic_location_object_get", 
"rna_ParticleHairKey_dynamic_location_object_set", NULL);
 }
 
 static void rna_def_particle_key(BlenderRNA *brna)

_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to