- Bugfix: 'Hide' button for PoseMode bones (in Editing Buttons) was setting the ...
authorJoshua Leung <aligorith@gmail.com>
Mon, 20 Oct 2008 04:53:49 +0000 (04:53 +0000)
committerJoshua Leung <aligorith@gmail.com>
Mon, 20 Oct 2008 04:53:49 +0000 (04:53 +0000)
- Tidied up comments in Action Editor code a bit

source/blender/src/buttons_editing.c
source/blender/src/editaction.c
source/blender/src/editaction_gpencil.c

index 4b3e8b1a05633f4e5cb36d50795682eae07425dc..dbf7e76d5f696b8e4065a393aa6496ad35b220a8 100644 (file)
@@ -4635,7 +4635,7 @@ static void editing_panel_pose_bones(Object *ob, bArmature *arm)
                        uiDefButBitI(block, TOG, BONE_NO_SCALE, B_ARM_RECALCDATA, "S",                  70,by-38,20,19, &curBone->flag, 1.0, 32.0, 0.0, 0.0, "Don't inherit scale from parent Bone");
                        uiDefButBitI(block, TOGN, BONE_NO_DEFORM, B_ARM_RECALCDATA, "Deform",   90, by-38, 80, 19, &curBone->flag, 0.0, 0.0, 0.0, 0.0, "Indicate if Bone deforms geometry");
                        uiDefButBitI(block, TOG, BONE_MULT_VG_ENV, B_ARM_RECALCDATA, "Mult",    170,by-38,80,19, &curBone->flag, 1.0, 32.0, 0.0, 0.0, "Multiply Bone Envelope with VertexGroup");
-                       uiDefButBitI(block, TOG, BONE_MULT_VG_ENV, B_ARM_RECALCDATA, "Hide",    250,by-38,80,19, &curBone->flag, 1.0, 32.0, 0.0, 0.0, "Toggles display of this bone in Edit Mode");
+                       uiDefButBitI(block, TOG, BONE_HIDDEN_P, B_ARM_RECALCDATA, "Hide",       250,by-38,80,19, &curBone->flag, 1.0, 32.0, 0.0, 0.0, "Toggles display of this bone in Edit Mode");
                        
                        /* layers */
                        uiBlockBeginAlign(block);
index 0f0e38084b62f5f40f08b59ff912249f3228e3ad..f976244bf916863d51ad2193d31a16a7940a32eb 100644 (file)
@@ -2095,9 +2095,9 @@ void paste_actdata ()
        short datatype;
        Object *ob= OBACT;
        
-       short no_name= 0;
-       float offset = CFRA - actcopy_firstframe;
+       const float offset = (float)(CFRA - actcopy_firstframe);
        char *actname = NULL, *conname = NULL;
+       short no_name= 0;
        
        /* check if buffer is empty */
        if (ELEM(NULL, actcopybuf.first, actcopybuf.last)) {
@@ -2118,13 +2118,13 @@ void paste_actdata ()
        
        /* from selected channels */
        for (ale= act_data.first; ale; ale= ale->next) {
-               Ipo *ipo_src=NULL;
+               Ipo *ipo_src = NULL;
                bActionChannel *achan;
                IpoCurve *ico, *icu;
                BezTriple *bezt;
                int i;
                
-               /* find matching ipo-block */
+               /* find suitable IPO-block from buffer to paste from */
                for (achan= actcopybuf.first; achan; achan= achan->next) {
                        /* try to match data */
                        if (ale->ownertype == ACTTYPE_ACHAN) {
@@ -2170,7 +2170,8 @@ void paste_actdata ()
                
                /* loop over curves, pasting keyframes */
                for (ico= ipo_src->curve.first; ico; ico= ico->next) {
-                       icu= verify_ipocurve((ID*)ob, ico->blocktype, actname, conname, NULL, ico->adrcode, 1);
+                       /* get IPO-curve to paste to (IPO-curve might not exist for destination, so gets created) */
+                       icu= verify_ipocurve((ID *)ob, ico->blocktype, actname, conname, NULL, ico->adrcode, 1);
                        
                        if (icu) {
                                /* just start pasting, with the the first keyframe on the current frame, and so on */
index 5ea12faffd444bb14b982841e563a31869768f1b..5d1e73a7b2862f7e3c84efbd35e9b3b1ef28b35c 100644 (file)
@@ -83,7 +83,7 @@
 #include "blendef.h"
 #include "butspace.h"
 
-#include "PIL_time.h"                  /* sleep                                */
+#include "PIL_time.h"
 #include "mydevice.h"
 
 /* ***************************************** */
@@ -220,17 +220,22 @@ void set_gplayer_frame_selection (bGPDlayer *gpl, short mode)
        select_gpencil_frames (gpl, mode);
 }
 
+/* select the frame in this layer that occurs on this frame (there should only be one at most) */
 void select_gpencil_frame (bGPDlayer *gpl, int selx, short select_mode)
 {
        bGPDframe *gpf;
    
        /* search through frames for a match */
        for (gpf= gpl->frames.first; gpf; gpf= gpf->next) {
-               if (gpf->framenum == selx)
+               /* there should only be one frame with this frame-number */
+               if (gpf->framenum == selx) {
                        gpframe_select(gpf, select_mode);
+                       break;
+               }
        }
 }
 
+/* select the frames in this layer that occur within the bounds specified */
 void borderselect_gplayer_frames (bGPDlayer *gpl, float min, float max, short select_mode)
 {
        bGPDframe *gpf;
@@ -290,6 +295,7 @@ void deselect_gpencil_layers (void *data, short mode)
 /* ***************************************** */
 /* Frame Editing Tools */
 
+/* Delete selected grease-pencil layers */
 void delete_gpencil_layers (void)
 {
        ListBase act_data = {NULL, NULL};
@@ -396,7 +402,7 @@ void free_gpcopybuf ()
 }
 
 /* This function adds data to the copy/paste buffer, freeing existing data first
- * Only the selected action channels gets their selected keyframes copied.
+ * Only the selected GP-layers get their selected keyframes copied.
  */
 void copy_gpdata ()
 {