commit 45a82a133c0f8d91bce6621be1ea2f4818e8e340
author Andy McFadden <fadden@android.com> Tue Jun 22 10:34:21 2010 -0700
committer Andy McFadden <fadden@android.com> Tue Jun 22 10:34:21 2010 -0700
tree 08ea280d27202edf15990f0f76d3b7603068b682
parent 364f9d924cbd9d392744a66f80cc084c3d80caf0

Fix dexdump.

It was printing bad output or crashing on the -wide-volatile instructions
generated by "dx".  This inverts the "else" clause, treating new-array
and instance-of as the special case.

Change-Id: I1717508d2e41cdc9a2396e2eaa215d7f9f8ac21b

dexdump/DexDump.c

diff --git a/dexdump/DexDump.c b/dexdump/DexDump.c
index e8dcf61..9582c98 100644
--- a/dexdump/DexDump.c
+++ b/dexdump/DexDump.c
@@ -836,7 +836,14 @@
             pDecInsn->vA, pDecInsn->vB, (s4)pDecInsn->vC, (u2)pDecInsn->vC);
         break;
     case kFmt22c:        // op vA, vB, thing@CCCC
-        if (pDecInsn->opCode >= OP_IGET && pDecInsn->opCode <= OP_IPUT_SHORT) {
+        if (pDecInsn->opCode == OP_INSTANCE_OF ||
+            pDecInsn->opCode == OP_NEW_ARRAY)
+        {
+            printf(" v%d, v%d, %s // class@%04x",
+                pDecInsn->vA, pDecInsn->vB,
+                getClassDescriptor(pDexFile, pDecInsn->vC), pDecInsn->vC);
+        } else {
+            /* iget* and iput*, including dexopt-generated -volatile */
             FieldMethodInfo fieldInfo;
             if (getFieldInfo(pDexFile, pDecInsn->vC, &fieldInfo)) {
                 printf(" v%d, v%d, %s.%s:%s // field@%04x", pDecInsn->vA,
@@ -846,10 +853,6 @@
                 printf(" v%d, v%d, ??? // field@%04x", pDecInsn->vA,
                     pDecInsn->vB, pDecInsn->vC);
             }
-        } else {
-            printf(" v%d, v%d, %s // class@%04x",
-                pDecInsn->vA, pDecInsn->vB,
-                getClassDescriptor(pDexFile, pDecInsn->vC), pDecInsn->vC);
         }
         break;
     case kFmt22cs:       // [opt] op vA, vB, field offset CCCC