Skip to content

Conversation

@rampage-raw
Copy link

Using [] in lua doesn't access the characters of a string, which results in the value of an array of type bool always being false

@google-cla
Copy link

google-cla bot commented Aug 16, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-actions github-actions bot added the lua label Aug 16, 2023
@le-michael
Copy link
Contributor

Can you add a test case to tests/luatest.lua for this change?

@rampage-raw
Copy link
Author

Can you add a test case to tests/luatest.lua for this change?

There are still some bugs in this fix, I'll take care of them later and add test cases.

@serprex
Copy link

serprex commented Mar 11, 2024

@Cindlory would you like me to pick up this PR & add a test?

There's also an issue with how bools are encoded in general, addressed in #8252

@rampage-raw
Copy link
Author

@Cindlory would you like me to pick up this PR & add a test?

There's also an issue with how bools are encoded in general, addressed in #8252

@serprex no problem. Due to project changes at work, I put this issue on hold. My PR may not fix the problem, I remember it may be related to Lua's generated code.

serprex added a commit to serprex/flatbuffers that referenced this pull request Mar 11, 2024
@github-actions
Copy link
Contributor

This pull request is stale because it has been open 6 months with no activity. Please comment or label not-stale, or this will be closed in 14 days.

@github-actions github-actions bot added the stale label Sep 24, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2024

This pull request was automatically closed due to no activity for 6 months plus the 14 day notice period.

@github-actions github-actions bot closed this Oct 9, 2024
serprex added a commit to PeerDB-io/peerdb that referenced this pull request Feb 13, 2025
generating correct lua code involved using my fork of flatbuffers,
I thought fixes would get merged upstream but flatbuffers maintainers ignored me

google/flatbuffers#8071
google/flatbuffers#8251
google/flatbuffers#8252
google/flatbuffers#8254
google/flatbuffers#8258
google/flatbuffers#8262
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants