Skip to content

Adding additional APIs and pointing header directly into file #2

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

traetox
Copy link

@traetox traetox commented Apr 19, 2021

Purpose of this PR:

  1. Updates the header to have a magic value so that we can identify the file
  2. Updates the header structure to point directly into the memory mapped file so that binary read/write cycles are not needed
  3. Added additional API to allow for overwriting pushes (NOT the default)
  4. Added Flock to unix builds
  5. Added additional open method that does not truncate allowing for the buffer to be opened and closed without overwriting header
  6. Added explicit close method so that systems can open and close this type resource

The PR should be backwards compatible and not break any existing usage, but does add some nice helpers and a little safety around opening existing buffers and making it possible to hand an existing buffer off to an external application which can consume it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants