The TriNavMesh.build() operation is not linking cells correctly.

[GoogleCodeExporter]

Cell linking in meshes created via TriNavMesh.build() is not complete.

The cause is the following code:

for (int iLink = 0; iLink < maxLinks; iLink++)
{
    if (cell.getLink(iLink) == null)
    {
        for (TriCell nCell : neighborCells)
        {
            if (cell.link(nCell, true) != TriCell.NULL_INDEX)
                // Successful link.
                break;
        }
    }
}

As a minimum, the getLink() check is inappropriate and needs to be removed.

The code should be re-designed to loop through the neighbor list and
exit early if link count reaches 3.

Original issue reported on code.google.com by steve...@gmail.com on 6 Jul 2010 at 8:37

[GoogleCodeExporter]

Original comment by steve...@gmail.com on 9 Jul 2010 at 4:07

• Added labels: Impact-High
• Removed labels: Priority-Critical

[GoogleCodeExporter]

Fixed in R78.  Passed junit tests.

Original comment by steve...@gmail.com on 9 Jul 2010 at 4:10

• Changed state: Fixed

[GoogleCodeExporter]

Fix deployed in cai-nav-0.1.2

Original comment by steve...@gmail.com on 10 Jul 2010 at 7:07

• Changed state: Deployed