swordlegend / recastnavigation

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

rcAssert/dtAssert warnings in gcc with -Wunused-value enabled #192

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
When compiling with gcc with -Wunused-value,

rcAssert/dtAssert will generate a warning similar to:

../../../3rdparty/recastnavigation/Detour/Source/DetourNavMesh.cpp: In member 
function `const dtOffMeshConnection* 
dtNavMesh::getOffMeshConnectionByRef(dtPolyRef) const':
../../../3rdparty/recastnavigation/Detour/Source/DetourNavMesh.cpp:1360: 
warning: left-hand operand of comma has no effect [-Wunused-value]

Original issue reported on code.google.com by mends...@gmail.com on 15 Feb 2012 at 2:23

GoogleCodeExporter commented 9 years ago
Fixed this a while back for our internal project but never got around to 
sending a fix.  Simply requires explicitly ignoring the result of the comparison

See: 
https://github.com/mendsley/recastnavigation/commit/329699ee56c98c4ff9130d2d49a4
7eae444ad953

Original comment by mends...@gmail.com on 15 Feb 2012 at 2:23

GoogleCodeExporter commented 9 years ago
Fixed in R335

Original comment by memono...@gmail.com on 10 Mar 2012 at 8:53