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.

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

Update LZ4 library to 1.9.2 #3415

Closed
wants to merge 2 commits into from
Closed

Update LZ4 library to 1.9.2 #3415

wants to merge 2 commits into from

Conversation

miguelportilla
Copy link
Contributor

  • Update the LZ4 library from 1.8.2 to 1.9.2
  • Use LZ4_decompress_safe instead of LZ4_decompress_fast which is unsafe and deprecated.

@miguelportilla miguelportilla requested review from gregtatcam, HowardHinnant and a user May 27, 2020 19:37
ghost
ghost approved these changes May 28, 2020
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Compiled and tested.

@ghost ghost mentioned this pull request May 28, 2020
Closed
@manojsdoshi manojsdoshi mentioned this pull request May 29, 2020
Closed
@carlhua carlhua added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label May 29, 2020
This was referenced May 29, 2020
Closed
Merged
@miguelportilla miguelportilla deleted the lz4 branch June 2, 2020 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gregtatcam gregtatcam gregtatcam approved these changes

@HowardHinnant HowardHinnant HowardHinnant approved these changes

Assignees

@gregtatcam gregtatcam

@HowardHinnant HowardHinnant

Labels
Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@miguelportilla @gregtatcam @HowardHinnant @carlhua