davebaol

Yeah I see your point. Not sure but considering [checkGuard occurrences](https://github.com/libgdx/gdx-ai/search?utf8=%E2%9C%93&q=checkGuard) it looks like you can replace code like this ````java child.setControl(this); child.start(); if (child.checkGuard(this)) child.run(); else child.fail(); ```` with...

Well, maybe I'm missing something. As said above, I've not tested this change myself. Just looked quickly into the code. Assuming you have a tree as simple as ```` parent...

Sounds reasonable. It looks like this code ````java child.setControl(this); child.start(); if (child.checkGuard(this)) child.run(); else child.fail(); ```` should become ````java child.setControl(this); if (child.checkGuard(this)) { child.start(); child.run(); } else childFail(null); // actually...

Hmmm... Shouldn't something like ``` toTarget.set(target.getPosition()).sub(owner.getPosition()); ``` be affected too?

At a first glance I'd say that `toTarget` should be an instance of a class that extends Vector2 and overrides the appropriate methods. But currently there's no way to set...

What exactly is the issue addressed by this PR?

Unfortunately, I won't have the time to investigate in the short run, sorry. PR welcome :)

Glad to see you back again. I'm going to investigate this issue asap. Thanks for the report.

Not sure how to reproduce your issue, but after a quick glance at the stack trace I guess that replacing the loop [do { ... } while (openList.size > 0);](https://github.com/libgdx/gdx-ai/blob/master/gdx-ai/src/com/badlogic/gdx/ai/pfa/indexed/IndexedAStarPathFinder.java#L137-L163)...

Well, yes, I have to admit that the documentation about interruptible pathfinding is not exhaustive at all. You should not call `PathFinderRequest.search()` directly. That method should be called for you...