From dfb5baf79b1e00b164088ec803b5a436c6eaa56c Mon Sep 17 00:00:00 2001 From: MihailRis Date: Sat, 31 Aug 2024 19:58:41 +0300 Subject: [PATCH] fix StructLayout::getNumber --- src/data/StructLayout.cpp | 3 ++- test/data/StructLayout.cpp | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) 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; });