google / google/flatbuffers

Make property renaming optional

Open
#7,111 84 comments 4 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
26.5k
Forks
3.7k
PR merge metrics
No merged PRs in 30d

Description

I am integrating Flatbuffers with [existing data standards](https://spacedatastandards.org/), it is mandatory to use the exact property names / namespaces contained in these standards going forward, including all capitalization.

I've hacked in a quick fix to remove [`MakeCamel`](https://github.com/google/flatbuffers/blob/master/src/idl_parser.cpp#L98), [`MakeScreamingCamel`](https://github.com/google/flatbuffers/blob/master/src/idl_parser.cpp#L112), and [`MakeCamelCase`](https://github.com/google/flatbuffers/blob/master/src/bfbs_gen.h#L96), as well as disabling this [warning](https://github.com/google/flatbuffers/blob/master/src/idl_parser.cpp#L753). I can work on a pull that puts disabling these behind a flag, for users in my situation, if it is something that would be accepted.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.