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

Update from 2.0.2 to 2.1.6 #16

Closed
wants to merge 799 commits into from
Closed

Update from 2.0.2 to 2.1.6 #16

wants to merge 799 commits into from

Conversation

al13n321
Copy link
Member

To fix ClickHouse/ClickHouse#63319 (just updating to latest version, didn't look for the exact commit that fixed it)

ccawley2011 and others added 30 commits January 22, 2023 00:58
make sure input/output args match original functions from clang/gcc
* Ubuntu 22.04 use different format for ABI files so old ones need to be removed
* Use more recent zlib-ng commit to avoid issues with internal adler32 and crc32 functions
* Add detection of XSAVE intrinsics
* Disable visibility check for Cygwin, MinGW and MSYS as the compiler will only issue warning instead of error for unsupported attributes.
A fixed block could be chosen when a stored block was smaller. Now
the smaller of the two is always chosen.
memLevel 9 would cause deflateBound() to assume the use of fixed
blocks, even if the compression level was 0, which forces stored
blocks. That could result in a bound less than the size of the
compressed data. Now level 0 always uses the stored blocks bound.
gzsetparams() now returns a Z_STREAM_ERROR in this case.
`functable` is already declared by functable.h which is included by functable.c
…mark#1454.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>
This should reduce the cost of indirection that occurs when calling functable
chunk copying functions inside inflate_fast. It should also allow the compiler
to optimize the inflate fast path for the specific architecture.
…tations.

Added debug assert check for value = 0.
Added more details to the comment to avoid future confusion.
Added fallback logic for older MSVC versions, just in case.
phprus and others added 25 commits December 21, 2023 16:12
… pointers from functable

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>
Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>
Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>
Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>
This requires minimum CMake 3.5 and C++11, this matches nicely with zlib-ng 2.1.x requirements.
Makefiles require using tabs as indents. So we need to remove this rule.
@CLAassistant
Copy link

CLAassistant commented May 27, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 14 committers have signed the CLA.

✅ al13n321
❌ Xeonacid
❌ mtl1979
❌ bebuch
❌ autoantwort
❌ Dead2
❌ yosh-matsuda
❌ phprus
❌ nmoinvaz
❌ KungFuJesus
❌ FantasqueX
❌ ccawley2011
❌ RyeMutt
❌ uilianries
You have signed the CLA already but the status is still pending? Let us recheck it.

@al13n321 al13n321 marked this pull request as ready for review May 28, 2024 19:02
@al13n321
Copy link
Member Author

#17

@al13n321 al13n321 closed this Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet