Tweener の _updateTask で "Uncaught RangeError: Maximum call stack size exceeded"

[fukuyama]

tweener を使うときに以下の条件で_updateTask() の無限再帰呼び出しが発生します。

• call か set のみの _tasks で、loop フラグを true にする
• loop フラグを true にしたまま、tweener.clear() のみを呼び出す

[simiraaaa]

callとsetのみはTweenerではなくupdateかenterframeイベントを使用
clearはタスクをクリアするだけなので、アニメーションを止める場合はpauseを使用

これで大丈夫じゃないかと思います

[simiraaaa]

clearはニュアンス的にアニメーション無くすような感じなのでclearでloopにfalseセットすれば、clearの方は大丈夫ですね

[fukuyama]

たしかに call / set のみの tweener は、使う側で気を付けるのはありかも。wait いれれば、動かすこともできるし、
MenuDialog は、enterframe でやってみて、問題なさそうでした。

loop を clear で、初期化してないのは、何か理由があるのかな～？とか思ってました^^
クリアしても問題ないですよね。たぶん。

clear しなくても _tasks が空で、loop == ture にすると、やっぱりひっかるので…
_updateTask の loop 部分で、_index を確認すれば防げると思うけど、どうでしょうか？

            var task = this._tasks[this._index];
            if (!task) {

                if (this.loop === true && this._index > 0) {
                    this._index = 0;
                    return this._updateTask(app);
                }
                else {
                    this.isPlaying = false;
                }

                return ;
            }

[simiraaaa]

そこでチェック入れてもいいですが、普通に考えても何もアニメーション追加せずにsetLoop(true)するのはおかしいですよねw

clearは意図的にアニメーションクリアするので、こっちでloop=falseするのは入れといたほうがいいですね。
clearにこれがないのは確か、loopをあとから追加したからだと思います。

[fukuyama]

なんとなく、どんなふうに使っても致命的なエラーには、ならないようにしておいた方が使う人にとっては良いかな～と思うところもあるんですよね。初心者な人とか特に。
でも、あまりガチガチにチェックして処理が重くなっても嫌だし…ってことで、このくらいなら良いかな？と

call / set の方は、うまい解決方法が思いつかない、使う人が注意で 😄

[simiraaaa]

call、setは一応、一度でもcallとset以外のアニメーションがセットされたかどうかのフラグを作ってそれでチェックすればできると思います。（このフラグはclearで消さないといけない)

[fukuyama]

MenuDialog の方は、enterfarme にして、プルリクします。