diff options
author | jackpoz <giacomopoz@gmail.com> | 2014-07-30 20:22:32 +0200 |
---|---|---|
committer | jackpoz <giacomopoz@gmail.com> | 2014-07-30 20:22:32 +0200 |
commit | 4190d84c5b4a9eb7d3d70f8872bba16b89bd1c09 (patch) | |
tree | 59d3a04d4ae3ce4a14c46f4b0f13e80233fb3870 /src | |
parent | 74d11ca0ef3eefb57121350cdf7614a7a46294f4 (diff) |
Tools/MMaps: Fix static analysis issues
Fix static analysis issue spotted by Coverity "Structurally dead code found"
Diffstat (limited to 'src')
-rw-r--r-- | src/tools/mmaps_generator/MapBuilder.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/tools/mmaps_generator/MapBuilder.cpp b/src/tools/mmaps_generator/MapBuilder.cpp index 3437355c0a9..b119b6cdbde 100644 --- a/src/tools/mmaps_generator/MapBuilder.cpp +++ b/src/tools/mmaps_generator/MapBuilder.cpp @@ -782,12 +782,12 @@ namespace MMAP if (params.nvp > DT_VERTS_PER_POLYGON) { printf("%s Invalid verts-per-polygon value! \n", tileString); - continue; + break; } if (params.vertCount >= 0xffff) { printf("%s Too many vertices! \n", tileString); - continue; + break; } if (!params.vertCount || !params.verts) { @@ -796,7 +796,7 @@ namespace MMAP // message is an annoyance //printf("%sNo vertices to build tile! \n", tileString); - continue; + break; } if (!params.polyCount || !params.polys || TILES_PER_MAP*TILES_PER_MAP == params.polyCount) @@ -805,19 +805,19 @@ namespace MMAP // keep in mind that we do output those into debug info // drop tiles with only exact count - some tiles may have geometry while having less tiles printf("%s No polygons to build on tile! \n", tileString); - continue; + break; } if (!params.detailMeshes || !params.detailVerts || !params.detailTris) { printf("%s No detail mesh to build tile! \n", tileString); - continue; + break; } printf("%s Building navmesh tile...\n", tileString); if (!dtCreateNavMeshData(¶ms, &navData, &navDataSize)) { printf("%s Failed building navmesh tile! \n", tileString); - continue; + break; } dtTileRef tileRef = 0; @@ -828,7 +828,7 @@ namespace MMAP if (!tileRef || dtResult != DT_SUCCESS) { printf("%s Failed adding tile to navmesh! \n", tileString); - continue; + break; } // file output @@ -841,7 +841,7 @@ namespace MMAP sprintf(message, "[Map %03i] Failed to open %s for writing!\n", mapID, fileName); perror(message); navMesh->removeTile(tileRef, NULL, NULL); - continue; + break; } printf("%s Writing to file...\n", tileString); |