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:
Julian Eisel 2016-09-18 21:54:24 +02:00
parent 572bc1364c
commit 404a427dfd
2 changed files with 6 additions and 6 deletions

View File

@ -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)
{

View File

@ -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) {