Skip to content

Conversation

Kn99HN
Copy link
Contributor

@Kn99HN Kn99HN commented Feb 7, 2023

Step 1 of Feature for #7687.

  • Create a File interface to write to file or save only file names.
  • Add a flag to indicate which file impl to use.

@github-actions github-actions bot added the c++ label Feb 7, 2023
@github-actions github-actions bot added the codegen Involving generating code from schema label Feb 9, 2023
Copy link

@tungmeoo tungmeoo left a comment

Choose a reason for hiding this comment

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

@Kn99HN Kn99HN marked this pull request as ready for review February 14, 2023 17:02
@Kn99HN Kn99HN changed the title WIP: Add a FileWriter interface Add a FileWriter interface Feb 15, 2023
@Kn99HN Kn99HN requested a review from dbaileychess February 27, 2023 06:33
@dbaileychess
Copy link
Collaborator

Ping @Kn99HN

@Kn99HN Kn99HN requested a review from dbaileychess March 18, 2023 17:09
dbaileychess
dbaileychess previously approved these changes Mar 24, 2023
@Kn99HN Kn99HN requested a review from dbaileychess March 31, 2023 03:59
@dbaileychess dbaileychess enabled auto-merge (squash) April 5, 2023 22:30
@dbaileychess dbaileychess merged commit 0916f1c into google:master Apr 6, 2023
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
* Add a FileWriter interface

* Change interface

* Provide 2 impl for File interface: FileManager & FileNameManager

* Update

* update

* Update

* Add file_writer file

* Update

* Format files

* Update based on review

* Update

* Format bzl file

* Add LoadFile function

* Format

---------

Co-authored-by: Derek Bailey <derekbailey@google.com>
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
* Add a FileWriter interface

* Change interface

* Provide 2 impl for File interface: FileManager & FileNameManager

* Update

* update

* Update

* Add file_writer file

* Update

* Format files

* Update based on review

* Update

* Format bzl file

* Add LoadFile function

* Format

---------

Co-authored-by: Derek Bailey <derekbailey@google.com>
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.

5 participants