[MOAB-dev] r4942 - MOAB/trunk/src/parallel

[hongjun at mcs.anl.gov]

Author: hongjun
Date: 2011-06-03 13:32:18 -0500 (Fri, 03 Jun 2011)
New Revision: 4942

Modified:
   MOAB/trunk/src/parallel/ParallelComm.cpp
Log:
o Fixed the misuse of MB_ALREADY_ALLOCATED checking in "tag_get_handle"
o Passes make check


Modified: MOAB/trunk/src/parallel/ParallelComm.cpp
===================================================================
--- MOAB/trunk/src/parallel/ParallelComm.cpp	2011-06-03 15:45:09 UTC (rev 4941)
+++ MOAB/trunk/src/parallel/ParallelComm.cpp	2011-06-03 18:32:18 UTC (rev 4942)
@@ -2510,7 +2510,7 @@
     std::vector<int> id_data(n_sets);
     result = mbImpl->tag_get_handle("PARALLEL_UNIQUE_ID", 1, MB_TYPE_INTEGER, 
                                 uid_tag, MB_TAG_SPARSE|MB_TAG_CREAT);
-    if (MB_ALREADY_ALLOCATED != result && MB_SUCCESS != result) {
+    if (MB_SUCCESS != result) {
       RRA("Trouble creating parallel geometry unique id tag.");
     }
     result = mbImpl->tag_get_data(uid_tag, all_sets, &id_data[0]);
@@ -2662,10 +2662,10 @@
     std::vector<int> uids(n_uid);
     UNPACK_INTS(buff_ptr, &uids[0], n_uid);
 
-    Tag uid_tag; int def_val = 0;
+    Tag uid_tag;
     result = mbImpl->tag_get_handle("PARALLEL_UNIQUE_ID", 1, MB_TYPE_INTEGER,
-                                    uid_tag, MB_TAG_SPARSE|MB_TAG_CREAT, &def_val);
-    if (MB_ALREADY_ALLOCATED != result && MB_SUCCESS != result) {
+                                    uid_tag, MB_TAG_SPARSE|MB_TAG_CREAT);
+    if (MB_SUCCESS != result) {
       RRA("Trouble creating parallel geometry unique id tag.");
     }