Minor corrections for previous commit
Was using wrong argument name in doxygen comment. Also reduced scope of vars. Sorry for the noise :/
This commit is contained in:
parent
572bc1364c
commit
404a427dfd
|
@ -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)
|
||||
{
|
||||
|
|
|
@ -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) {
|
||||
|
|
Loading…
Reference in New Issue