[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [35997] trunk/blender/source/blender/ makesrna/intern/rna_particle.c: Fix for [#26747] Crash accessing dynamic hair RNA

Janne Karhu jhkarh at gmail.com
Mon Apr 4 15:47:28 CEST 2011


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)




More information about the Bf-blender-cvs mailing list