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

Minimal in-place edit support #9

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

Minimal in-place edit support #9

wants to merge 15 commits into from

Conversation

neilpa
Copy link
Owner

@neilpa neilpa commented Apr 5, 2020

Adds minimal support for in-place edit. Requires that the updated payload be the exact size of the segment data it's replacing for now.

Initial attempt to resolve #4 (also resolves #6).

@neilpa neilpa changed the title Minimal in-place edit suppor Minimal in-place edit support Apr 5, 2020
neilpa added 2 commits April 5, 2020 16:07
Adds the ability to append new segments right before the SOS segment.
Uses xio.Splice to handle that actual file manipulation which works on a
temp copy of the file to avoid corrupting the original before moving it
back.
This is a quick tool to exercise inserting new JFIF segments and that it
works as expected.
@neilpa neilpa added this to the v1 milestone May 8, 2020
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.

Document usage and example(s) in README Segment update API
1 participant