[MOAB-dev] r5341 - in MOAB/trunk/src: . parallel

[tautges at mcs.anl.gov]

Author: tautges
Date: 2012-01-23 15:31:41 -0600 (Mon, 23 Jan 2012)
New Revision: 5341

Modified:
   MOAB/trunk/src/DualTool.cpp
   MOAB/trunk/src/Skinner.cpp
   MOAB/trunk/src/SmoothFace.cpp
   MOAB/trunk/src/parallel/ParallelComm.cpp
   MOAB/trunk/src/parallel/WriteHDF5Parallel.cpp
Log:
Fixing an assert with a paste-o (WriteHDF5Parallel.cpp) and a number of
compiler warnings (others).



Modified: MOAB/trunk/src/DualTool.cpp
===================================================================
--- MOAB/trunk/src/DualTool.cpp	2012-01-23 18:35:17 UTC (rev 5340)
+++ MOAB/trunk/src/DualTool.cpp	2012-01-23 21:31:41 UTC (rev 5341)
@@ -123,10 +123,8 @@
                                    MB_TAG_SPARSE|MB_TAG_CREAT, 
                                    &dum_int);
   assert(MB_SUCCESS == result);
-
-    // empty statement to avoid warning
-  (void)(result);
-
+  if (MB_SUCCESS == result); // empty statement to get rid of warning.
+  
   maxHexId = -1;
 }
 

Modified: MOAB/trunk/src/Skinner.cpp
===================================================================
--- MOAB/trunk/src/Skinner.cpp	2012-01-23 18:35:17 UTC (rev 5340)
+++ MOAB/trunk/src/Skinner.cpp	2012-01-23 21:31:41 UTC (rev 5341)
@@ -1219,7 +1219,6 @@
   // intersection with the input range and the edges adjacent to a vertex.
 
   ErrorCode rval;
-  std::vector<EntityHandle>::iterator i;
   Range::iterator hint = skin_verts.begin();
   
   // All input entities must be edges.

Modified: MOAB/trunk/src/SmoothFace.cpp
===================================================================
--- MOAB/trunk/src/SmoothFace.cpp	2012-01-23 18:35:17 UTC (rev 5340)
+++ MOAB/trunk/src/SmoothFace.cpp	2012-01-23 21:31:41 UTC (rev 5341)
@@ -1116,8 +1116,6 @@
 
   double big_dist = compareTol * 1.0e3;
 
-  bool done = false; // maybe not use this
-
   // from the list of close facets, determine the closest point
   for (size_t i = 0; i < facet_list.size(); i++)
   {
@@ -1159,7 +1157,6 @@
 
       if (dist < compareTol)
       {
-        done = true;
         break;
       }