From 40d07f09a45b2d13188dbde557444fa0baff1d35 Mon Sep 17 00:00:00 2001
From: Kawe Mazidjatari <48657826+Mauler125@users.noreply.github.com>
Date: Sun, 11 Aug 2024 22:11:52 +0200
Subject: [PATCH] Recast: set tile cell default height to tile's bmax

A more sane fallback value vs setting it to 0.
---
 src/thirdparty/recast/Detour/Source/DetourNavMeshBuilder.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/thirdparty/recast/Detour/Source/DetourNavMeshBuilder.cpp b/src/thirdparty/recast/Detour/Source/DetourNavMeshBuilder.cpp
index 4f078c96..495af421 100644
--- a/src/thirdparty/recast/Detour/Source/DetourNavMeshBuilder.cpp
+++ b/src/thirdparty/recast/Detour/Source/DetourNavMeshBuilder.cpp
@@ -598,7 +598,7 @@ bool createPolyMeshCells(const dtNavMeshCreateParams* params, rdTempVector<CellI
 				float targetCellPos[3];
 				targetCellPos[0] = params->bmin[0]+j*stepX+offsetX;
 				targetCellPos[1] = params->bmin[1]+k*stepY;
-				targetCellPos[2] = 0; // todo(amos): might need a proper fallback, but so far this never failed.
+				targetCellPos[2] = params->bmax[2]; // note(amos): might need a better fallback, but so far this never failed.
 
 				if (!rdPointInPolygon(targetCellPos, polyVerts, nv))
 					continue;