diff --git a/src/data/StructLayout.cpp b/src/data/StructLayout.cpp
index 417bc6e7..01a89d2d 100644
--- a/src/data/StructLayout.cpp
+++ b/src/data/StructLayout.cpp
@@ -310,7 +310,8 @@ number_t StructLayout::getNumber(
         case FieldType::I64:
         case FieldType::CHAR:
             return getInteger(src, name, index);
-        
+        default:
+            throw std::runtime_error("type error");
     }
     throw std::runtime_error("type error");
 }
diff --git a/test/data/StructLayout.cpp b/test/data/StructLayout.cpp
index 720434ef..bf6f8e1d 100644
--- a/test/data/StructLayout.cpp
+++ b/test/data/StructLayout.cpp
@@ -92,6 +92,8 @@ TEST(StructLayout, ConvertWithLoss) {
     };
     auto dstLayout = StructLayout::create(dstFields);
     auto report = srcLayout.checkCompatibility(dstLayout);
+
+    // check report
     std::sort(report.begin(), report.end(), [](const auto& a, const auto& b) {
         return a.name < b.name;
     });