Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use switch statements for BASE_TYPE_ lookups #7813

Merged
merged 1 commit into from
Feb 5, 2023

Conversation

dbaileychess
Copy link
Collaborator

Since enums are not defined in enum-value order, some of the mapping to arrays were incorrect. Just switch them out for switch statements.

This is not in any performance related area, so going from index to switch statement should not matter.

@github-actions github-actions bot added c++ codegen Involving generating code from schema labels Feb 5, 2023
@dbaileychess dbaileychess merged commit f3a3f45 into google:master Feb 5, 2023
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Oct 16, 2024
This change is a bug fix in the automatically generated code that was introduced by the new version of the flatbuffer generator that TensorFlow updated to in c17d64d which includes the following change google/flatbuffers#7813 which fixed the underlying flatbuffer code generator bug.

PiperOrigin-RevId: 685943323
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Oct 16, 2024
This change is a bug fix in the automatically generated code that was introduced by the new version of the flatbuffer generator that TensorFlow updated to in c17d64d which includes the following change google/flatbuffers#7813 which fixed the underlying flatbuffer code generator bug.

PiperOrigin-RevId: 685943323
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Oct 16, 2024
This change is a bug fix in the automatically generated code that was introduced by the new version of the flatbuffer generator that TensorFlow updated to in c17d64d which includes the following change google/flatbuffers#7813 which fixed the underlying flatbuffer code generator bug.

PiperOrigin-RevId: 686567841
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Oct 16, 2024
This change is a bug fix in the automatically generated code that was introduced by the new version of the flatbuffer generator that TensorFlow updated to in c17d64d which includes the following change google/flatbuffers#7813 which fixed the underlying flatbuffer code generator bug.

PiperOrigin-RevId: 686164263
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ codegen Involving generating code from schema
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant