Skip to content

Version 3.1 #25

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.

Sign up for GitHub

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

Jump to bottom
Draft
TheZoc wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Version 3.1 #25

TheZoc wants to merge 9 commits into from

Conversation

@TheZoc
Copy link
Owner

@TheZoc TheZoc commented Jun 2, 2022 •
edited
Loading

  • Build a separate slim library that supports only patch application

Smaller tasks that went in:

  • Removed custom allocators. We now use the ones supplied by Alloc.h, which includes the Big allocator. This should allow a faster operation when allocating for larger files and the OS provides (e.g. VirtualAlloc on Windows)
  • Simplified and cleaned up the log system
  • Simplified and cleared up the CreatePatch() and ApplyPatch() function calls. There are now Ex function variants to allow for more customization
  • CreatePatch() now normalizes the patches passed to it internally
  • Removed a few auxiliary functions from the header files - they are now static to the cpp file

@TheZoc TheZoc self-assigned this Jun 2, 2022
@TheZoc TheZoc force-pushed the version3.1 branch 4 times, most recently from 7364c73 to 880cc4d Compare June 5, 2022 03:47
TheZoc added 2 commits June 5, 2022 05:05
@TheZoc
- GetDifferences(), CreatePatch() and DoCreatePatch() now handle the …
e8eec5c 
…path normalization internally

- Simplified CreatePatch() parameters
- Added CreatePatchEx()
- Added GetDifferencesEx()
- DoCreatePatch() and PrintCreatePatchProgressBar() are now static and were removed from the header
- Updated CreatePatch and VisualCreatePatch to match the changes
- PatchFileList_t is now a struct
@TheZoc
- DoApplyPatchFile() and RestoreBackup() are now static and were remo…
65012e0 
…ved from the header file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@TheZoc TheZoc

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TheZoc