Possibly incorrect out of bounds index check in DetourCrowd.cpp

What steps will reproduce the problem?
1. Just noticed this in code, didn't try to create a repro case.

What is the expected output? What do you see instead?
In DetourCrowd.cpp, there are some instances of the following lines of code: 
if (idx < 0 || idx > m_maxAgents)
    return false;
e.g. requestMoveTargetReplan(), requestMoveTarget(), requestMoveVelocity(), 
resetMoveTarget()

Should the > condition be >= instead?

What version of the product are you using? On what operating system?
Exists in the latest version.

Please provide any additional information below.
-

Original issue reported on code.google.com by loonych...@gmail.com on 2 Nov 2012 at 11:16

swordlegend / recastnavigation

Possibly incorrect out of bounds index check in DetourCrowd.cpp #221