ActionManager memory leak

cocos2d-x3.7

---

都3.7了还有这样的bug，真是好难过，不过还是好开源的，谁都可以贡献一下

问题描述：

当创建一个node，并让它run一个action的时，如果这个node没有添加到任何其它节点（即没有父节点），那么改node和action将一直不会被释放。

---

原因：

在node调用runAction时，会将动画交给ActionManager来管理，即调用ActionManager::addAction，看该函数的实现：

void ActionManager::addAction(Action *action, Node *target, bool paused)

{

    //...

    if (! element)

    {

        element = (tHashElement*)calloc(sizeof(*element), 1);

        element->paused = paused;

        target->retain();   //!!!watch out!!!

        element->target = target;

        HASH_ADD_PTR(_targets, target, element);

    }

	//...

注意这里会把node给retain一次，那它是在哪里release的呢，看下面：

void ActionManager::update(float dt)

{

	//...

	elt = (tHashElement*)(elt->hh.next);

	if (_currentTargetSalvaged && _currentTarget->actions->num == 0)

	{

		deleteHashElement(_currentTarget);

	}

	//...

}

void ActionManager::deleteHashElement(tHashElement *element)

{

    ccArrayFree(element->actions);

    HASH_DEL(_targets, element);

    element->target->release();   // release node

    free(element);

}

也就是说，当一个node在运行一个action时会被retain（referenceCount+1），在action运行完成后就会被release（referenceCount-1）。这么看起来没什么问题，但是当这个node不在running状态时，即没有添加到任何节点，通过create创建下一帧应该被自动释放，如果这时候给它运行了一个动画（ActionManager::addAction时第三个参数paused为true），再看update函数：

void ActionManager::update(float dt)

{

	if (! _currentTarget->paused)  //动画不会被执行

	{

		//...

	}

	//...

	elt = (tHashElement*)(elt->hh.next);

	if (_currentTargetSalvaged && _currentTarget->actions->num == 0)

	{

		deleteHashElement(_currentTarget);

	}

	//...

}

此时动画一直不会执行，也就是说动画不会完成，那么actions->num不可能等于0，则该节点就一直不会被释放。

---

解决方法：

void ActionManager::update(float dt)

{

	//...

	elt = (tHashElement*)(elt->hh.next);

	if (_currentTargetSalvaged && _currentTarget->actions->num == 0)

	{

		deleteHashElement(_currentTarget);

	}

	//if some node refrence 'target', it's refrence count >= 2 (issues #14050)

	else if (_currentTarget->target->getReferenceCount() == 1)

	{

		deleteHashElement(_currentTarget);

	}

	//...

}

那么问题来了：为什么会有这样的行为呢——创建一个node，不加到任何节点，还给它run一个action？

这里解释过一遍了：https://github.com/cocos2d/cocos2d-x/pull/14090