Bia10 / recastnavigation

Automatically exported from code.google.com/p/recastnavigation
zlib License
1 stars 0 forks source link

Creating a navmesh with no unique detail verts causes a crash #213

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
I am attempting to convert Blender/BGE from version 1.4 of Recast and Detour to 
the SVN version. I seem to have most everything working again, but I am having 
trouble with detailed meshes.

Here is the original code: http://www.pasteall.org/34682/cpp
Here is my new version: http://www.pasteall.org/34681/cpp

The old code could handle a ndvertsuniq value of 0 (and consequently a NULL 
dvertices), but dtCreateNavMEshData() will crash given the same information (it 
attempts to memcpy from dvertices on line 562 of DetourNavMeshBuilder.cpp).

I am still new to the library (the developer that originally did the 
integration has stopped doing Blender/BGE development), so it is possible that 
I could just be messing this up.

Original issue reported on code.google.com by mogurijin on 24 Aug 2012 at 7:10

GoogleCodeExporter commented 8 years ago
The latest version allows you to leave the detailmesh empty, try that. That is, 
just leave all the detailmesh params to zero.

Original comment by memono...@gmail.com on 27 Aug 2012 at 7:13

GoogleCodeExporter commented 8 years ago
At the moment that is what I'm doing; I just left them empty by commenting out 
lines 86 to 90. I'm not noticing any difference, but I'm not entirely sure what 
the detailed mesh stuff does. However, since the old code used them, I thought 
I should use them to make sure nothing  breaks.

Original comment by mogurijin on 27 Aug 2012 at 7:36

GoogleCodeExporter commented 8 years ago
Detail tris are used to add extra extra height definition for the navmesh 
polygons. Your vode serms to just ognore it, so it is safe to set them to null.

Original comment by memono...@gmail.com on 29 Aug 2012 at 9:13

GoogleCodeExporter commented 8 years ago

Original comment by memono...@gmail.com on 16 Sep 2013 at 7:10