Skip to content

Conversation

@kou
Copy link

@kou kou commented Nov 20, 2025

This supports only reading FlatBuffers data.

I confirmed that this implementation works for
reflection/reflection.bfbs and tests/monsterdata_test.mon. This includes tests for them in tests/ruby/. tests/RubyTest.rb is a test runner.

Other .rb files in tests/ are auto generated files by scripts/generate_code.py. I followed the suggestion in the pull request template.

I'll work on the following tasks after this is merged:

  • Add support for writing FlatBuffers data.
  • Prepare RubyGems package.

@github-actions github-actions bot added c++ CI Continuous Integration codegen Involving generating code from schema python labels Nov 20, 2025
namespace {

static void ForAllEnums(
static inline void ForAllEnums(
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

static inlines in this file is for suppressing unused-function warnings. This Ruby implementation doesn't use IsVector(). If we use static not static inline, unused-function warning is emitted for this Ruby implementation.

@kou kou force-pushed the ruby branch 2 times, most recently from 9984b13 to fee1978 Compare November 23, 2025 15:04
This supports only reading FlatBuffers data.

I confirmed that this implementation works for
`reflection/reflection.bfbs` and `tests/monsterdata_test.mon`. This
includes tests for them in `tests/ruby/`. `tests/RubyTest.rb` is a
test runner.

Other `.rb` files in `tests/` are auto generated files by
`scripts/generate_code.py`. I followed the suggestion in the pull
request template.

I'll work on the following tasks after this is merged:

* Add support for writing FlatBuffers data.
* Prepare RubyGems package.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ CI Continuous Integration codegen Involving generating code from schema python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant