cocos2dx - ActionManager内存泄露

cocos2d-x3.7 ActionManager memory leak

都3.7了还有这样的bug，有点悲伤，不过还是好开源的，谁都可以贡献一下。

已经给官方提了PullRequest，详情参见：https://github.com/cocos2d/cocos2d-x/pull/14090

问题描述：

当创建一个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);
	}
	//...
}