]> git.ipfire.org Git - thirdparty/valgrind.git/commitdiff
drd: Source code cleanup - change two variables from int to bool and outdent one...
authorBart Van Assche <bvanassche@acm.org>
Sun, 3 Jul 2011 11:40:49 +0000 (11:40 +0000)
committerBart Van Assche <bvanassche@acm.org>
Sun, 3 Jul 2011 11:40:49 +0000 (11:40 +0000)
git-svn-id: svn://svn.valgrind.org/valgrind/trunk@11848

drd/drd_thread.c

index a5abaf6af0e770d3ec4d08e03a557d36ae02a2e8..7b833ef892232da379604cd782c52300dc5aeb4b 100644 (file)
@@ -1509,10 +1509,10 @@ void DRD_(thread_update_conflict_set)(const DrdThreadId tid,
 
       for (q = DRD_(g_threadinfo)[j].last; q; q = q->prev)
       {
-         const int included_in_old_conflict_set
+         const Bool included_in_old_conflict_set
             = ! DRD_(vc_lte)(&q->vc, old_vc)
             && ! DRD_(vc_lte)(old_vc, &q->vc);
-         const int included_in_new_conflict_set
+         const Bool included_in_new_conflict_set
             = ! DRD_(vc_lte)(&q->vc, new_vc)
             && ! DRD_(vc_lte)(new_vc, &q->vc);
          if (included_in_old_conflict_set != included_in_new_conflict_set)
@@ -1546,19 +1546,17 @@ void DRD_(thread_update_conflict_set)(const DrdThreadId tid,
    DRD_(bm_clear_marked)(DRD_(g_conflict_set));
 
    p = DRD_(g_threadinfo)[tid].last;
+   for (j = 0; j < DRD_N_THREADS; j++)
    {
-      for (j = 0; j < DRD_N_THREADS; j++)
+      if (j != tid && DRD_(IsValidDrdThreadId)(j))
       {
-         if (j != tid && DRD_(IsValidDrdThreadId)(j))
+         Segment* q;
+         for (q = DRD_(g_threadinfo)[j].last; q; q = q->prev)
          {
-            Segment* q;
-            for (q = DRD_(g_threadinfo)[j].last; q; q = q->prev)
+            if (! DRD_(vc_lte)(&q->vc, &p->vc)
+                && ! DRD_(vc_lte)(&p->vc, &q->vc))
             {
-               if (! DRD_(vc_lte)(&q->vc, &p->vc)
-                   && ! DRD_(vc_lte)(&p->vc, &q->vc))
-               {
-                  DRD_(bm_merge2_marked)(DRD_(g_conflict_set), DRD_(sg_bm)(q));
-               }
+               DRD_(bm_merge2_marked)(DRD_(g_conflict_set), DRD_(sg_bm)(q));
             }
          }
       }