commit b2e78d39f6df19a6222b3cd4c7010864fb8cb55f
author Carl Shapiro <cshapiro@google.com> Fri Aug 20 11:34:18 2010 -0700
committer Carl Shapiro <cshapiro@google.com> Wed Sep 01 18:32:40 2010 -0700
tree 7c9c24bff9976a1449e1d519c1941f971f17bc27
parent 5d81aa327a11f6e929c2396d34c343923268bdc3

Consistently refer to the mark context as ctx.

vm/alloc/MarkSweep.c

diff --git a/vm/alloc/MarkSweep.c b/vm/alloc/MarkSweep.c
index a5d31b5..eaf73c7 100644
--- a/vm/alloc/MarkSweep.c
+++ b/vm/alloc/MarkSweep.c
@@ -95,13 +95,13 @@
 bool
 dvmHeapBeginMarkStep(GcMode mode)
 {
-    GcMarkContext *mc = &gDvm.gcHeap->markContext;
+    GcMarkContext *ctx = &gDvm.gcHeap->markContext;
 
-    if (!createMarkStack(&mc->stack)) {
+    if (!createMarkStack(&ctx->stack)) {
         return false;
     }
-    mc->finger = NULL;
-    mc->immuneLimit = dvmHeapSourceGetImmuneLimit(mode);
+    ctx->finger = NULL;
+    ctx->immuneLimit = dvmHeapSourceGetImmuneLimit(mode);
     return true;
 }
 
@@ -758,14 +758,14 @@
  */
 void dvmHandleSoftRefs(Object **list)
 {
-    GcMarkContext *markContext;
+    GcMarkContext *ctx;
     Object *ref, *referent;
     Object *clear;
     size_t referentOffset;
     size_t counter;
     bool marked;
 
-    markContext = &gDvm.gcHeap->markContext;
+    ctx = &gDvm.gcHeap->markContext;
     referentOffset = gDvm.offJavaLangRefReference_referent;
     clear = NULL;
     counter = 0;
@@ -773,10 +773,10 @@
         ref = dequeuePendingReference(list);
         referent = dvmGetFieldObject(ref, referentOffset);
         assert(referent != NULL);
-        marked = isMarked(referent, markContext);
+        marked = isMarked(referent, ctx);
         if (!marked && ((++counter) & 1)) {
             /* Referent is white and biased toward saving, mark it. */
-            markObject(referent, markContext);
+            markObject(referent, ctx);
             marked = true;
         }
         if (!marked) {
@@ -789,7 +789,7 @@
      * Restart the mark with the newly black references added to the
      * root set.
      */
-    processMarkStack(markContext);
+    processMarkStack(ctx);
 }
 
 /*
@@ -799,19 +799,19 @@
  */
 void dvmClearWhiteRefs(Object **list)
 {
-    GcMarkContext *markContext;
+    GcMarkContext *ctx;
     Object *ref, *referent;
     size_t referentOffset;
     bool doSignal;
 
-    markContext = &gDvm.gcHeap->markContext;
+    ctx = &gDvm.gcHeap->markContext;
     referentOffset = gDvm.offJavaLangRefReference_referent;
     doSignal = false;
     while (*list != NULL) {
         ref = dequeuePendingReference(list);
         referent = dvmGetFieldObject(ref, referentOffset);
         assert(referent != NULL);
-        if (!isMarked(referent, markContext)) {
+        if (!isMarked(referent, ctx)) {
             /* Referent is white, clear it. */
             clearReference(ref);
             if (isEnqueuable(ref)) {
@@ -840,7 +840,7 @@
     Object **ref;
     Object **lastRef;
     size_t totalPendCount;
-    GcMarkContext *markContext = &gDvm.gcHeap->markContext;
+    GcMarkContext *ctx = &gDvm.gcHeap->markContext;
 
     /*
      * All reachable objects have been marked.
@@ -871,7 +871,7 @@
         gapRef = ref = finRefs->refs.table;
         lastRef = finRefs->refs.nextEntry;
         while (ref < lastRef) {
-            if (!isMarked(*ref, markContext)) {
+            if (!isMarked(*ref, ctx)) {
                 if (!dvmHeapAddToHeapRefTable(&newPendingRefs, *ref)) {
                     //TODO: add the current table and allocate
                     //      a new, smaller one.
@@ -932,19 +932,19 @@
     HPROF_SET_GC_SCAN_STATE(HPROF_ROOT_FINALIZING, 0);
     while (ref < lastRef) {
         assert(*ref != NULL);
-        markObject(*ref, markContext);
+        markObject(*ref, ctx);
         ref++;
     }
     HPROF_CLEAR_GC_SCAN_STATE();
-    processMarkStack(markContext);
+    processMarkStack(ctx);
     dvmSignalHeapWorker(false);
 }
 
 void dvmHeapFinishMarkStep()
 {
-    GcMarkContext *markContext;
+    GcMarkContext *ctx;
 
-    markContext = &gDvm.gcHeap->markContext;
+    ctx = &gDvm.gcHeap->markContext;
 
     /* The mark bits are now not needed.
      */
@@ -952,9 +952,9 @@
 
     /* Clean up everything else associated with the marking process.
      */
-    destroyMarkStack(&markContext->stack);
+    destroyMarkStack(&ctx->stack);
 
-    markContext->finger = NULL;
+    ctx->finger = NULL;
 }
 
 typedef struct {