Remove the offset table

getml · Jan 19, 2025 · 0dd56d0 · 0dd56d0
1 parent 88421cb
commit 0dd56d0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 14 deletions.
diff --git a/include/rfl/flatbuf/schema/Type.hpp b/include/rfl/flatbuf/schema/Type.hpp
@@ -1,8 +1,6 @@
 #ifndef RFL_FLATBUF_SCHEMA_TYPE_HPP_
 #define RFL_FLATBUF_SCHEMA_TYPE_HPP_
 
-#include <flatbuffers/flatbuffers.h>
-
 #include <iostream>
 #include <map>
 #include <memory>
@@ -67,13 +65,8 @@ struct Type {
   };
 
   struct Table {
-    struct Field {
-      std::string name;
-      rfl::Ref<Type> type;
-      flatbuffers::uoffset_t offset;
-    };
     std::string name;
-    std::vector<Field> fields;
+    std::vector<std::pair<std::string, Type>> fields;
   };
 
   struct Union {

diff --git a/src/rfl/flatbuf/Type.cpp b/src/rfl/flatbuf/Type.cpp
@@ -126,7 +126,7 @@ std::ostream& operator<<(std::ostream& _os, const Type::Table& _t) {
   _os << "table " << internal::strings::to_pascal_case(_t.name) << " {"
       << std::endl;
   for (const auto& f : _t.fields) {
-    _os << " " << f.name << ":" << *f.type << ";" << std::endl;
+    _os << " " << f.first << ":" << f.second << ";" << std::endl;
   }
   return _os << "}" << std::endl;
 }

diff --git a/src/rfl/flatbuf/schema/internal_schema_to_flatbuf_schema.cpp b/src/rfl/flatbuf/schema/internal_schema_to_flatbuf_schema.cpp
@@ -107,11 +107,9 @@ Type object_to_flatbuf_schema_type(
     const bool _is_top_level, FlatbufTypes* _flatbuf_types) {
   Type::Table table_schema;
   for (const auto& [k, v] : _obj.types_) {
-    table_schema.fields.push_back(Type::Table::Field{
-        .name = k,
-        .type = rfl::Ref<Type>::make(type_to_flatbuf_schema_type(
-            v, _definitions, false, _flatbuf_types)),
-        .offset = 0 /* TODO*/});
+    table_schema.fields.push_back(std::make_pair(
+        k,
+        type_to_flatbuf_schema_type(v, _definitions, false, _flatbuf_types)));
   }
   if (_is_top_level) {
     return Type{.value = table_schema};