Dep/Recast: Fixed area type merges

TrinityCore · Apr 21, 2018 · a4892d6 · a4892d6
1 parent 577466b
commit a4892d6
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 4 deletions.
diff --git a/dep/recastnavigation/Recast/Source/RecastRasterization.cpp b/dep/recastnavigation/Recast/Source/RecastRasterization.cpp
@@ -122,16 +122,16 @@ static bool addSpan(rcHeightfield& hf, const int x, const int y,
 		}
 		else
 		{
+			// Merge flags.
+			if (rcAbs((int)s->smax - (int)cur->smax) <= flagMergeThr)
+				s->area = rcMax(s->area, cur->area);
+
 			// Merge spans.
 			if (cur->smin < s->smin)
 				s->smin = cur->smin;
 			if (cur->smax > s->smax)
 				s->smax = cur->smax;
 
-			// Merge flags.
-			if (rcAbs((int)s->smax - (int)cur->smax) <= flagMergeThr)
-				s->area = rcMax(s->area, cur->area);
-
 			// Remove current span.
 			rcSpan* next = cur->next;
 			freeSpan(hf, cur);

diff --git a/dep/recastnavigation/recastnavigation_2_area_merge.diff b/dep/recastnavigation/recastnavigation_2_area_merge.diff
@@ -0,0 +1,25 @@
+diff --git a/dep/recastnavigation/Recast/Source/RecastRasterization.cpp b/dep/recastnavigation/Recast/Source/RecastRasterization.cpp
+index a4cef74909..2ffffae8b0 100644
+--- a/dep/recastnavigation/Recast/Source/RecastRasterization.cpp
++++ b/dep/recastnavigation/Recast/Source/RecastRasterization.cpp
+@@ -122,16 +122,16 @@ static bool addSpan(rcHeightfield& hf, const int x, const int y,
+ 		}
+ 		else
+ 		{
++			// Merge flags.
++			if (rcAbs((int)s->smax - (int)cur->smax) <= flagMergeThr)
++				s->area = rcMax(s->area, cur->area);
++
+ 			// Merge spans.
+ 			if (cur->smin < s->smin)
+ 				s->smin = cur->smin;
+ 			if (cur->smax > s->smax)
+ 				s->smax = cur->smax;
+
+-			// Merge flags.
+-			if (rcAbs((int)s->smax - (int)cur->smax) <= flagMergeThr)
+-				s->area = rcMax(s->area, cur->area);
+-			
+ 			// Remove current span.
+ 			rcSpan* next = cur->next;
+ 			freeSpan(hf, cur);