[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [34863] trunk/blender/source/blender/ editors/armature/poselib.c: PoseLib: Adding support for custom properties
Joshua Leung
aligorith at gmail.com
Tue Feb 15 12:47:10 CET 2011
Revision: 34863
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=34863
Author: aligorith
Date: 2011-02-15 11:47:09 +0000 (Tue, 15 Feb 2011)
Log Message:
-----------
PoseLib: Adding support for custom properties
Custom properties are now supported by Pose Library. They will get
saved and restored correctly during previewing and pose adding.
To do this, I've changed PoseLib over from using the "LocRotScale"
Builtin Keying Set to the "Whole Character" one instead. One
consequence of this, is that now entire poses are stored in the Pose
Lib, irrespective of whether you only selected part of a pose to store
(i.e. facial controls only, or hand poses only). If this is enough of
a problem, I'll make a second Keying Set that does take selections
into account, and use that here instead.
Modified Paths:
--------------
trunk/blender/source/blender/editors/armature/poselib.c
Modified: trunk/blender/source/blender/editors/armature/poselib.c
===================================================================
--- trunk/blender/source/blender/editors/armature/poselib.c 2011-02-15 11:43:35 UTC (rev 34862)
+++ trunk/blender/source/blender/editors/armature/poselib.c 2011-02-15 11:47:09 UTC (rev 34863)
@@ -49,6 +49,7 @@
#include "BKE_action.h"
#include "BKE_armature.h"
#include "BKE_depsgraph.h"
+#include "BKE_idprop.h"
#include "BKE_context.h"
#include "BKE_report.h"
@@ -264,11 +265,6 @@
/* ************************************************************* */
-/* Pointers to the builtin KeyingSets that we want to use */
-static KeyingSet *poselib_ks_locrotscale = NULL; /* the only keyingset we'll need */
-
-/* ----- */
-
static void poselib_add_menu_invoke__replacemenu (bContext *C, uiLayout *layout, void *UNUSED(arg))
{
Object *ob= ED_object_pose_armature(CTX_data_active_object(C));
@@ -335,6 +331,7 @@
bArmature *arm= (ob) ? ob->data : NULL;
bPose *pose= (ob) ? ob->pose : NULL;
TimeMarker *marker;
+ KeyingSet *ks= ANIM_builtin_keyingset_get_named(NULL, "Whole Character"); /* this includes custom props :)*/
int frame= RNA_int_get(op->ptr, "frame");
char name[64];
@@ -367,12 +364,10 @@
/* validate name */
BLI_uniquename(&act->markers, marker, "Pose", '.', offsetof(TimeMarker, name), sizeof(marker->name));
- /* KeyingSet to use depends on rotation mode (but that's handled by the templates code) */
- if (poselib_ks_locrotscale == NULL)
- poselib_ks_locrotscale= ANIM_builtin_keyingset_get_named(NULL, "LocRotScale");
-
- /* make the keyingset use context info to determine where to add keyframes */
- ANIM_apply_keyingset(C, NULL, act, poselib_ks_locrotscale, MODIFYKEY_MODE_INSERT, (float)frame);
+ /* use Keying Set to determine what to store for the pose */
+ // FIXME: in the past, the Keying Set respected selections (LocRotScale), but the current one doesn't (Whole Character)
+ // so perhaps we need either a new Keying Set, or just to add overrides here...
+ ANIM_apply_keyingset(C, NULL, act, ks, MODIFYKEY_MODE_INSERT, (float)frame);
/* store new 'active' pose number */
act->active_marker= BLI_countlist(&act->markers);
@@ -646,8 +641,10 @@
typedef struct tPoseLib_Backup {
struct tPoseLib_Backup *next, *prev;
- bPoseChannel *pchan;
- bPoseChannel olddata;
+ bPoseChannel *pchan; /* pose channel backups are for */
+
+ bPoseChannel olddata; /* copy of pose channel's old data (at start) */
+ IDProperty *oldprops; /* copy (needs freeing) of pose channel's properties (at start) */
} tPoseLib_Backup;
/* Makes a copy of the current pose for restoration purposes - doesn't do constraints currently */
@@ -671,6 +668,9 @@
plb->pchan= pchan;
memcpy(&plb->olddata, plb->pchan, sizeof(bPoseChannel));
+ if (pchan->prop)
+ plb->oldprops= IDP_CopyProperty(pchan->prop);
+
BLI_addtail(&pld->backups, plb);
/* mark as being affected */
@@ -681,16 +681,42 @@
}
}
-/* Restores original pose - doesn't do constraints currently */
+/* Restores original pose */
static void poselib_backup_restore (tPoseLib_PreviewData *pld)
{
tPoseLib_Backup *plb;
for (plb= pld->backups.first; plb; plb= plb->next) {
+ /* copy most of data straight back */
memcpy(plb->pchan, &plb->olddata, sizeof(bPoseChannel));
+
+ /* just overwrite values of properties from the stored copies (there should be some) */
+ if (plb->oldprops)
+ IDP_SyncGroupValues(plb->pchan->prop, plb->oldprops);
+
+ // TODO: constraints settings aren't restored yet, even though these could change (though not that likely)
}
}
+/* Free list of backups, including any side data it may use */
+static void poselib_backup_free_data (tPoseLib_PreviewData *pld)
+{
+ tPoseLib_Backup *plb, *plbn;
+
+ for (plb= pld->backups.first; plb; plb= plbn) {
+ plbn= plb->next;
+
+ /* free custom data */
+ if (plb->oldprops) {
+ IDP_FreeProperty(plb->oldprops);
+ MEM_freeN(plb->oldprops);
+ }
+
+ /* free backup element now */
+ BLI_freelinkN(&pld->backups, plb);
+ }
+}
+
/* ---------------------------- */
/* Applies the appropriate stored pose from the pose-library to the current pose
@@ -762,6 +788,10 @@
bAction *act= pld->act;
bActionGroup *agrp;
+ KeyingSet *ks = ANIM_get_keyingset_for_autokeying(scene, "Whole Character");
+ ListBase dsources = {NULL, NULL};
+ short autokey = autokeyframe_cfra_can_key(scene, &pld->ob->id);
+
/* start tagging/keying */
for (agrp= act->groups.first; agrp; agrp= agrp->next) {
/* only for selected action channels */
@@ -769,20 +799,9 @@
pchan= get_pose_channel(pose, agrp->name);
if (pchan) {
- if (autokeyframe_cfra_can_key(scene, &pld->ob->id)) {
- ListBase dsources = {NULL, NULL};
-
- /* get KeyingSet to use */
- KeyingSet *ks = ANIM_get_keyingset_for_autokeying(scene, "LocRotScale");
-
- /* now insert the keyframe(s) using the Keying Set
- * 1) add datasource override for the PoseChannel
- * 2) insert keyframes
- * 3) free the extra info
- */
+ if (autokey) {
+ /* add datasource override for the PoseChannel, to be used later */
ANIM_relative_keyingset_add_source(&dsources, &pld->ob->id, &RNA_PoseBone, pchan);
- ANIM_apply_keyingset(C, &dsources, NULL, ks, MODIFYKEY_MODE_INSERT, (float)CFRA);
- BLI_freelistN(&dsources);
/* clear any unkeyed tags */
if (pchan->bone)
@@ -797,6 +816,13 @@
}
}
+ /* perform actual auto-keying now */
+ if (autokey) {
+ /* insert keyframes for all relevant bones in one go */
+ ANIM_apply_keyingset(C, &dsources, NULL, ks, MODIFYKEY_MODE_INSERT, (float)CFRA);
+ BLI_freelistN(&dsources);
+ }
+
/* send notifiers for this */
WM_event_add_notifier(C, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
}
@@ -1358,8 +1384,8 @@
}
}
- /* free memory used for backups */
- BLI_freelistN(&pld->backups);
+ /* free memory used for backups and searching */
+ poselib_backup_free_data(pld);
BLI_freelistN(&pld->searchp);
/* free temp data for operator */
More information about the Bf-blender-cvs
mailing list