RecastDemo/Source/NavMeshTesterTool.cpp | 52 4 + 48 - 0 !
1 file changed, 4 insertions(+), 48 deletions(-)

 [patch 01/36] removed fixupcorridor and replaced with
 dtMergeCorridorStartMoved (#637)

* removed fixupCorridor and replaced function calls with dtMergeCorridorStartMoved

* removed fixupCorridor and replaced function calls with dtMergeCorridorStartMoved in NavMeshTesterTool.cpp

Fix #634

Recast/Source/RecastArea.cpp | 4 2 + 2 - 0 !
1 file changed, 2 insertions(+), 2 deletions(-)

 [patch 02/36] bounds checks were unnecessary inclusive. (#418)

Co-authored-by: Graham Pentheny <grahamboree@users.noreply.github.com>

Tests/Contrib/catch2/catch_all.hpp | 3911 2397 + 1514 - 0 !
Tests/Contrib/catch2/catch_amalgamated.cpp | 2212 1306 + 906 - 0 !
2 files changed, 3703 insertions(+), 2420 deletions(-)

 [patch 03/36] update catch to v3.4.0 (#648)

Recast/Include/Recast.h | 118 75 + 43 - 0 !
Recast/Source/RecastArea.cpp | 955 541 + 414 - 0 !
RecastDemo/Source/Sample_SoloMesh.cpp | 4 2 + 2 - 0 !
3 files changed, 618 insertions(+), 459 deletions(-)

 [patch 04/36] improved variable naming in recastarea.cpp (#636)

* Cleanup compact heightfield functions in RecastArea.cpp
* More improved variable names for RecastArea.cpp
* Improved variable names and documentation in rcOffsetPoly
* Don't normalize the miter bisector in rcOffsetPoly since this needs to be proportional to both segment normals.
* Moved vector normalization out of rcOffsetPoly into a helper function rcVsafeNormalize
* Rename local variables in rcOffsetPoly
* Rename BMiterX/Z to cornerMIterX/Z
* Also fixed some comment descriptions
* Added docstring for rcVsafeNormalize
* Improved clarity of a few comments

DetourCrowd/Source/DetourPathCorridor.cpp | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch 05/36] fix invalid assert dtassert(npath < m_maxpath) (#619)

.github/workflows/Tests.yaml | 6 3 + 3 - 0 !
1 file changed, 3 insertions(+), 3 deletions(-)

 [patch 06/36] enable verbose catch2 output when running tests in ci

Detour/Include/DetourNavMesh.h | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch 07/36] fix typo in dtpoly member documentation (#649)

Detour/Include/DetourNavMesh.h | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch 08/36] fix typo in detournavmesh.h (#651)

Detour/Include/DetourAssert.h | 2 1 + 1 - 0 !
Detour/Source/DetourAssert.cpp | 2 1 + 1 - 0 !
Integration.md | 32 32 + 0 - 0 !
README.md | 10 1 + 9 - 0 !
Recast/Include/RecastAssert.h | 2 1 + 1 - 0 !
Recast/Source/RecastAssert.cpp | 2 1 + 1 - 0 !
RecastDemo/premake5.lua | 2 1 + 1 - 0 !
7 files changed, 38 insertions(+), 14 deletions(-)

 [patch 09/36] changed asserts to be explicitly disabled with
 RC_DISABLE_ASSERTS rather than using NDEBUG (#653)

* Changed asserts to be explicitly disabled with RC_DISABLE_ASSERTS rather than using NDEBUG

This gives explicit control to users over when recast asserts are enabled or disabled.  This is useful when users wish to disable asserts in debug mode or enable them in release mode.  e.g. it's common to compile game code in debug alongside third party code in release.

* Added Integration.md with documentation on the integration process for Recast

Recast/Source/RecastRasterization.cpp | 81 40 + 41 - 0 !
1 file changed, 40 insertions(+), 41 deletions(-)

 [patch 10/36] some quick cleanup in recastrasterization.cpp (#655)

* Rename hf -> heightfield to be consistent
* Removed unused include

CONTRIBUTING.md | 4 2 + 2 - 0 !
Detour/Include/DetourNavMeshBuilder.h | 2 1 + 1 - 0 !
Detour/Include/DetourNavMeshQuery.h | 8 4 + 4 - 0 !
Detour/Source/DetourNavMeshBuilder.cpp | 2 1 + 1 - 0 !
Detour/Source/DetourNavMeshQuery.cpp | 26 13 + 13 - 0 !
DetourCrowd/Include/DetourCrowd.h | 4 2 + 2 - 0 !
DetourCrowd/Source/DetourObstacleAvoidance.cpp | 2 1 + 1 - 0 !
DetourTileCache/Include/DetourTileCacheBuilder.h | 2 1 + 1 - 0 !
README.md | 2 1 + 1 - 0 !
Recast/Include/Recast.h | 2 1 + 1 - 0 !
Recast/Include/RecastAlloc.h | 2 1 + 1 - 0 !
Recast/Source/RecastContour.cpp | 6 3 + 3 - 0 !
Recast/Source/RecastMesh.cpp | 2 1 + 1 - 0 !
Recast/Source/RecastMeshDetail.cpp | 4 2 + 2 - 0 !
Recast/Source/RecastRegion.cpp | 6 3 + 3 - 0 !
RecastDemo/Source/NavMeshPruneTool.cpp | 2 1 + 1 - 0 !
RecastDemo/Source/Sample_SoloMesh.cpp | 6 3 + 3 - 0 !
Tests/Recast/Tests_Recast.cpp | 4 2 + 2 - 0 !
18 files changed, 43 insertions(+), 43 deletions(-)

 [patch 11/36] typo fixes (#660)

* multiple comments typo fixes

* typo

* typos

* typos

* typos

Recast/Source/RecastMesh.cpp | 13 7 + 6 - 0 !
1 file changed, 7 insertions(+), 6 deletions(-)

 [patch 12/36] minor refactor in recastmesh.cpp (#661)

Recast/Source/RecastRegion.cpp | 7 6 + 1 - 0 !
1 file changed, 6 insertions(+), 1 deletion(-)

 [patch 13/36] fix:rcbuildlayerregions missing areatype (#470)

Co-authored-by: sunming <359944951@qq.com>
Co-authored-by: Graham Pentheny <grahamboree@users.noreply.github.com>

README.md | 4 3 + 1 - 0 !
1 file changed, 3 insertions(+), 1 deletion(-)

 [patch 14/36] added links in readme to docs site and github repo to
 make it easy to find one from the other. (#677)

Recast/Source/RecastMeshDetail.cpp | 64 38 + 26 - 0 !
1 file changed, 38 insertions(+), 26 deletions(-)

 [patch 15/36] make detail mesh edge detection more robust (#657)

Instead of using a distance check which can fail at large magnitudes due
to low precision we can check whether the edges are actually on the
hull.

Recast/Source/RecastFilter.cpp | 31 18 + 13 - 0 !
1 file changed, 18 insertions(+), 13 deletions(-)

 [patch 22/36] code cleanup and small optimizations in
 RecastFilter.cpp rcFilterLedgeSpans (#672)

* Code cleanup and minor refactor in RecastFilter.cpp rcFilterLedgeSpans

Because span.smax is always > 0, bot > 0 as well, and (-walkableClimb - bot) is always < -walkableClimb. Furthermore, as long as minNeighborHeight < -walkableClimb' at least once, there is no need to continue the traversal.

* Code cleanup and minor refactor in RecastFilter.cpp rcFilterLedgeSpans

Because span.smax is always > 0, bot > 0 as well, and (-walkableClimb - bot) is always < -walkableClimb. Furthermore, as long as minNeighborHeight < -walkableClimb' at least once, there is no need to continue the traversal.

* Update RecastFilter.cpp

Revise Comment

Recast/Include/Recast.h | 22 12 + 10 - 0 !
Recast/Source/RecastFilter.cpp | 32 15 + 17 - 0 !
Tests/Recast/Bench_rcVector.cpp | 168 168 + 0 - 0 !
Tests/Recast/Tests_Alloc.cpp | 260 260 + 0 - 0 !
Tests/Recast/Tests_Recast.cpp | 468 28 + 440 - 0 !
Tests/Recast/Tests_RecastFilter.cpp | 389 389 + 0 - 0 !
6 files changed, 872 insertions(+), 467 deletions(-)

 [patch 23/36] some unit tests for heightfield filtering functions
 (#682)

This adds some unit tests for the functions in RecastFilter.cpp, and updates docs around these functions. This also splits up the Tests_Recast.cpp file into a few smaller, more focused files.

Recast/Include/Recast.h | 3 2 + 1 - 0 !
Recast/Source/RecastFilter.cpp | 126 70 + 56 - 0 !
Tests/Recast/Tests_RecastFilter.cpp | 56 3 + 53 - 0 !
3 files changed, 75 insertions(+), 110 deletions(-)

 [patch 24/36] cleanup filter code and improved documentation (#683)

This mostly just changes variable names and adds some comments to make the code more clear.

It also has a few small fixup changes to the unit tests.

CHANGELOG.md | 52 49 + 3 - 0 !
1 file changed, 49 insertions(+), 3 deletions(-)

 [patch 25/36] backported old release notes to changelog.md for
 posterity

Docs/FAQ.md | 38 38 + 0 - 0 !
1 file changed, 38 insertions(+)

 [patch 26/36] ported faq to a markdown file in the docs folder

So that it's shipped with the code and also part of the doxygen docs at recastnav.com

CHANGELOG.md | 14 7 + 7 - 0 !
1 file changed, 7 insertions(+), 7 deletions(-)

 [patch 27/36] removed link brakets from old versions in changelog

Docs/_1_Introducation.md | 85 85 + 0 - 0 !
1 file changed, 85 insertions(+)

 [patch 28/36] added introduction documentation

Docs/Extern/Recast_api.txt | 8 4 + 4 - 0 !
1 file changed, 4 insertions(+), 4 deletions(-)

 [patch 31/36] fixed old parameter names in out-of-line recast
 function documentation

DetourCrowd/Source/DetourPathCorridor.cpp | 10 5 + 5 - 0 !
RecastDemo/premake5.lua | 3 2 + 1 - 0 !
Tests/CMakeLists.txt | 15 10 + 5 - 0 !
Tests/DetourCrowd/Tests_DetourPathCorridor.cpp | 119 119 + 0 - 0 !
4 files changed, 136 insertions(+), 11 deletions(-)

 [patch 32/36] fix out of bounds access in dtmergecorridorstartmoved
 and add tests (#635)

size can become negative if req > maxPath. This may happen when visited buffer is larger than path buffer.

CMakeLists.txt | 6 3 + 3 - 0 !
1 file changed, 3 insertions(+), 3 deletions(-)

 [patch 33/36] cmake: .pc: let gnuinstalldirs generate the paths
 (#692)

The concatenation was resulting in paths like
libdir=${exec_prefix}//nix/store/...-recastnavigation-1.6.0/lib

Detour/Include/DetourNavMeshBuilder.h | 4 2 + 2 - 0 !
Detour/Source/DetourNavMeshBuilder.cpp | 6 3 + 3 - 0 !
DetourTileCache/Include/DetourTileCacheBuilder.h | 2 1 + 1 - 0 !
3 files changed, 6 insertions(+), 6 deletions(-)

 [patch 35/36] fix: 6 typos (#698)

Signed-off-by: RoboSchmied <github@roboschmie.de>

CMakeLists.txt | 3 2 + 1 - 0 !
DebugUtils/CMakeLists.txt | 2 1 + 1 - 0 !
Detour/CMakeLists.txt | 2 1 + 1 - 0 !
DetourCrowd/CMakeLists.txt | 2 1 + 1 - 0 !
DetourTileCache/CMakeLists.txt | 2 1 + 1 - 0 !
Recast/CMakeLists.txt | 2 1 + 1 - 0 !
6 files changed, 7 insertions(+), 6 deletions(-)

---

CMakeLists.txt | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch] fix build with cmake 4 (#775)