commit 4b3fdb21a4e07b6f1d1b6753203703d7fdd9a785
author bart <bart@a5019735-40e9-0310-863c-91ae7b9d1cf9> Sun Jul 03 11:40:49 2011 +0000
committer bart <bart@a5019735-40e9-0310-863c-91ae7b9d1cf9> Sun Jul 03 11:40:49 2011 +0000
tree 1b491f49e037469da6dc07baec7e478c775a41e9
parent f5fe4b66b06998f8fa0eaa2bb17eb34eb0e3bbde

drd: Source code cleanup - change two variables from int to bool and outdent one code block

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@11848 a5019735-40e9-0310-863c-91ae7b9d1cf9

drd/drd_thread.c

diff --git a/drd/drd_thread.c b/drd/drd_thread.c
index a5abaf6..7b833ef 100644
--- a/drd/drd_thread.c
+++ b/drd/drd_thread.c
@@ -1509,10 +1509,10 @@
 
       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 @@
    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));
             }
          }
       }