From 191943d5c6b245f6550d70ca1d3f2498f44dc9a5 Mon Sep 17 00:00:00 2001
From: Sam Ansmink <samansmink@hotmail.com>
Date: Tue, 14 May 2024 15:10:42 +0200
Subject: [PATCH] Refactor initialization of EngineSchemaVisitor object

---
 src/include/delta_utils.hpp | 33 +++++++++++++++++----------------
 1 file changed, 17 insertions(+), 16 deletions(-)

diff --git a/src/include/delta_utils.hpp b/src/include/delta_utils.hpp
index 648b8d1..164ee69 100644
--- a/src/include/delta_utils.hpp
+++ b/src/include/delta_utils.hpp
@@ -15,22 +15,23 @@ class SchemaVisitor {
 
     static unique_ptr<FieldList> VisitSnapshotSchema(const ffi::SnapshotHandle* snapshot) {
         SchemaVisitor state;
-        ffi::EngineSchemaVisitor visitor = {
-                .data = &state,
-                .make_field_list = (uintptr_t (*)(void*, uintptr_t)) &MakeFieldList,
-                .visit_struct = (void (*)(void*, uintptr_t, ffi::KernelStringSlice, uintptr_t)) &VisitStruct,
-                .visit_string = VisitSimpleType<LogicalType::VARCHAR>(),
-                .visit_long = VisitSimpleType<LogicalType::BIGINT>(),
-                .visit_integer = VisitSimpleType<LogicalType::SMALLINT>(),
-                .visit_byte = VisitSimpleType<LogicalType::TINYINT>(),
-                .visit_float = VisitSimpleType<LogicalType::FLOAT>(),
-                .visit_double = VisitSimpleType<LogicalType::DOUBLE>(),
-                .visit_boolean = VisitSimpleType<LogicalType::BOOLEAN>(),
-                .visit_binary = VisitSimpleType<LogicalType::VARCHAR>(), // TODO: check
-                .visit_date = VisitSimpleType<LogicalType::DATE>(), // TODO: check
-                .visit_timestamp = VisitSimpleType<LogicalType::TIMESTAMP>(), // TODO: check
-                .visit_timestamp_ntz = VisitSimpleType<LogicalType::TIMESTAMP_TZ>() // TODO: check
-        };
+        ffi::EngineSchemaVisitor visitor;
+
+        visitor.data = &state;
+        visitor.make_field_list = (uintptr_t (*)(void*, uintptr_t)) &MakeFieldList;
+        visitor.visit_struct = (void (*)(void*, uintptr_t, ffi::KernelStringSlice, uintptr_t)) &VisitStruct;
+        visitor.visit_string = VisitSimpleType<LogicalType::VARCHAR>();
+        visitor.visit_long = VisitSimpleType<LogicalType::BIGINT>();
+        visitor.visit_integer = VisitSimpleType<LogicalType::SMALLINT>();
+        visitor.visit_byte = VisitSimpleType<LogicalType::TINYINT>();
+        visitor.visit_float = VisitSimpleType<LogicalType::FLOAT>();
+        visitor.visit_double = VisitSimpleType<LogicalType::DOUBLE>();
+        visitor.visit_boolean = VisitSimpleType<LogicalType::BOOLEAN>();
+        visitor.visit_binary = VisitSimpleType<LogicalType::VARCHAR>(); // TODO: check
+        visitor.visit_date = VisitSimpleType<LogicalType::DATE>(); // TODO: check
+        visitor.visit_timestamp = VisitSimpleType<LogicalType::TIMESTAMP>(); // TODO: check
+        visitor.visit_timestamp_ntz = VisitSimpleType<LogicalType::TIMESTAMP_TZ>(); // TODO: check
+
         uintptr_t result = visit_schema(snapshot, &visitor);
         return state.TakeFieldList(result);
     }