diff options
| author | Julian Eisel <eiseljulian@gmail.com> | 2016-09-18 22:54:24 +0300 |
|---|---|---|
| committer | Julian Eisel <eiseljulian@gmail.com> | 2016-09-18 22:54:24 +0300 |
| commit | 404a427dfdee0f5300b7c4f375cd7ff4a0b8bc8e (patch) | |
| tree | 2c062fcbb99d41687e434078f1141366af30d985 | |
| parent | 572bc1364ca9d978edf5aee991849dd4f8e56a52 (diff) | |
Minor corrections for previous commit
Was using wrong argument name in doxygen comment. Also reduced scope of vars.
Sorry for the noise :/
| -rw-r--r-- | source/blender/blenlib/intern/listbase.c | 6 | ||||
| -rw-r--r-- | source/blender/editors/armature/pose_group.c | 6 |
2 files changed, 6 insertions, 6 deletions
diff --git a/source/blender/blenlib/intern/listbase.c b/source/blender/blenlib/intern/listbase.c index e72eff57ad9..d6c0219ea23 100644 --- a/source/blender/blenlib/intern/listbase.c +++ b/source/blender/blenlib/intern/listbase.c @@ -343,12 +343,12 @@ void BLI_insertlinkbefore(ListBase *listbase, void *vnextlink, void *vnewlink) } /** - * Reinsert \a link relative to its current position but offset by \a step. Doesn't move + * Reinsert \a vlink relative to its current position but offset by \a step. Doesn't move * item if new position would exceed list (could optionally move to head/tail). * * \param step: Absolute value defines step size, sign defines direction. E.g pass -1 - * to move \a link before previous, or 1 to move behind next. - * \return If position of \a link has changed. + * to move \a vlink before previous, or 1 to move behind next. + * \return If position of \a vlink has changed. */ bool BLI_listbase_link_move(ListBase *listbase, void *vlink, int step) { diff --git a/source/blender/editors/armature/pose_group.c b/source/blender/editors/armature/pose_group.c index 85cff2e4df5..c492772b282 100644 --- a/source/blender/editors/armature/pose_group.c +++ b/source/blender/editors/armature/pose_group.c @@ -292,7 +292,6 @@ static int group_move_exec(bContext *C, wmOperator *op) bPoseChannel *pchan; bActionGroup *grp; int dir = RNA_enum_get(op->ptr, "direction"); - int grpIndexA, grpIndexB; if (ELEM(NULL, ob, pose)) return OPERATOR_CANCELLED; @@ -305,9 +304,10 @@ static int group_move_exec(bContext *C, wmOperator *op) return OPERATOR_CANCELLED; /* move bone group */ - grpIndexA = pose->active_group; if (BLI_listbase_link_move(&pose->agroups, grp, dir)) { - grpIndexB = grpIndexA + dir; + int grpIndexA = pose->active_group; + int grpIndexB = grpIndexA + dir; + pose->active_group += dir; /* fix changed bone group indices in bones (swap grpIndexA with grpIndexB) */ for (pchan = ob->pose->chanbase.first; pchan; pchan = pchan->next) { |