commit e2c57816b388e688caf0099cdf4aac91ae4008b6
author Mathieu Chartier <mathieuc@google.com> Fri Mar 06 21:11:22 2015 +0000
committer Android Git Automerger <android-git-automerger@android.com> Fri Mar 06 21:11:22 2015 +0000
tree 7afa976f6c13417e767925f844553dd1ca397eb9
parent 8bf66ca623c0d1f7742467b86bbd7daada0dda46
parent 3b5408b565509341301e6da82f3d07c75aad5f51

am 3b5408b5: Merge "Fix failing hprof DCHECK"

* commit '3b5408b565509341301e6da82f3d07c75aad5f51':
  Fix failing hprof DCHECK

runtime/hprof/hprof.cc

diff --git a/runtime/hprof/hprof.cc b/runtime/hprof/hprof.cc
index 7da22b1..5a7b7e1 100644
--- a/runtime/hprof/hprof.cc
+++ b/runtime/hprof/hprof.cc
@@ -1008,17 +1008,26 @@
       HprofBasicType t = SignatureToBasicTypeAndSize(f->GetTypeDescriptor(), &size);
       __ AddStringId(LookupStringId(f->GetName()));
       __ AddU1(t);
-      switch (size) {
-        case 1:
+      switch (t) {
+        case hprof_basic_byte:
           __ AddU1(f->GetByte(klass));
           break;
-        case 2:
+        case hprof_basic_boolean:
+          __ AddU1(f->GetBoolean(klass));
+          break;
+        case hprof_basic_char:
           __ AddU2(f->GetChar(klass));
           break;
-        case 4:
+        case hprof_basic_short:
+          __ AddU2(f->GetShort(klass));
+          break;
+        case hprof_basic_float:
+        case hprof_basic_int:
+        case hprof_basic_object:
           __ AddU4(f->Get32(klass));
           break;
-        case 8:
+        case hprof_basic_double:
+        case hprof_basic_long:
           __ AddU8(f->Get64(klass));
           break;
         default:
@@ -1099,16 +1108,29 @@
     for (int i = 0; i < ifieldCount; ++i) {
       mirror::ArtField* f = klass->GetInstanceField(i);
       size_t size;
-      SignatureToBasicTypeAndSize(f->GetTypeDescriptor(), &size);
-      if (size == 1) {
+      auto t = SignatureToBasicTypeAndSize(f->GetTypeDescriptor(), &size);
+      switch (t) {
+      case hprof_basic_byte:
         __ AddU1(f->GetByte(obj));
-      } else if (size == 2) {
+        break;
+      case hprof_basic_boolean:
+        __ AddU1(f->GetBoolean(obj));
+        break;
+      case hprof_basic_char:
         __ AddU2(f->GetChar(obj));
-      } else if (size == 4) {
+        break;
+      case hprof_basic_short:
+        __ AddU2(f->GetShort(obj));
+        break;
+      case hprof_basic_float:
+      case hprof_basic_int:
+      case hprof_basic_object:
         __ AddU4(f->Get32(obj));
-      } else {
-        CHECK_EQ(size, 8U);
+        break;
+      case hprof_basic_double:
+      case hprof_basic_long:
         __ AddU8(f->Get64(obj));
+        break;
       }
     }