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

Include win directory in pigz even if not using threads. #1035

Merged
merged 1 commit into from Sep 3, 2021

Conversation

nmoinvaz
Copy link
Member

@nmoinvaz nmoinvaz commented Aug 15, 2021

This PR always includes win directory in pigz when on Windows even if not using pthreads. This win directory contains source code the mimic the necessary linux functions. https://github.com/zlib-ng/pigzbench/tree/master/pigz/win
By not compiling with pthreads on Windows we can get to compiling pigz quicker for testing functional issues unrelated to threading.

@codecov
Copy link

codecov bot commented Aug 15, 2021

Codecov Report

Merging #1035 (73c58bd) into develop (e128537) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1035      +/-   ##
===========================================
- Coverage    78.32%   78.31%   -0.02%     
===========================================
  Files           85       85              
  Lines         8730     8730              
  Branches      1392     1392              
===========================================
- Hits          6838     6837       -1     
  Misses        1357     1357              
- Partials       535      536       +1     
Flag Coverage Δ
macos_clang 68.06% <ø> (ø)
macos_gcc 66.70% <ø> (ø)
ubuntu_clang 73.36% <ø> (ø)
ubuntu_clang_debug 73.68% <ø> (ø)
ubuntu_clang_inflate_allow_invalid_dist 73.11% <ø> (ø)
ubuntu_clang_inflate_strict 73.35% <ø> (ø)
ubuntu_clang_mmap 73.34% <ø> (ø)
ubuntu_clang_msan 73.36% <ø> (ø)
ubuntu_clang_pigz 34.08% <ø> (ø)
ubuntu_clang_pigz_no_optim 38.18% <ø> (ø)
ubuntu_clang_pigz_no_threads 33.62% <ø> (ø)
ubuntu_clang_reduced_mem 73.63% <ø> (ø)
ubuntu_gcc 72.83% <ø> (ø)
ubuntu_gcc_aarch64 74.74% <ø> (ø)
ubuntu_gcc_aarch64_compat_no_opt 72.38% <ø> (ø)
ubuntu_gcc_aarch64_no_acle 74.02% <ø> (ø)
ubuntu_gcc_aarch64_no_neon 74.29% <ø> (ø)
ubuntu_gcc_armhf 74.73% <ø> (ø)
ubuntu_gcc_armhf_compat_no_opt 72.35% <ø> (ø)
ubuntu_gcc_armhf_no_acle 74.99% <ø> (ø)
ubuntu_gcc_armhf_no_neon 75.23% <ø> (ø)
ubuntu_gcc_armsf 74.74% <ø> (ø)
ubuntu_gcc_armsf_compat_no_opt 72.35% <ø> (ø)
ubuntu_gcc_compat_no_opt 73.49% <ø> (ø)
ubuntu_gcc_mingw_i686 0.00% <ø> (ø)
ubuntu_gcc_mingw_x86_64 0.00% <ø> (ø)
ubuntu_gcc_no_avx2 74.71% <ø> (ø)
ubuntu_gcc_no_pclmulqdq 71.40% <ø> (ø)
ubuntu_gcc_no_sse2 73.81% <ø> (ø)
ubuntu_gcc_no_sse4 71.72% <ø> (ø)
ubuntu_gcc_o3 72.03% <ø> (ø)
ubuntu_gcc_osb 75.85% <ø> (ø)
ubuntu_gcc_pigz 34.43% <ø> (ø)
ubuntu_gcc_pigz_aarch64 36.15% <ø> (+0.03%) ⬆️
ubuntu_gcc_ppc 72.71% <ø> (ø)
ubuntu_gcc_ppc64 75.61% <ø> (ø)
ubuntu_gcc_ppc64le 74.15% <ø> (ø)
ubuntu_gcc_ppc_no_power8 75.25% <ø> (ø)
ubuntu_gcc_s390x 74.05% <ø> (ø)
ubuntu_gcc_sparc64 76.39% <ø> (ø)
win64_gcc ∅ <ø> (∅)
win64_gcc_compat_no_opt ∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
trees.c 96.17% <0.00%> (-0.28%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e128537...73c58bd. Read the comment docs.

Copy link
Member

@Dead2 Dead2 left a comment

Choose a reason for hiding this comment

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

LGTM

@nmoinvaz nmoinvaz added the cleanup Improving maintainability or removing code. label Aug 26, 2021
@Dead2 Dead2 merged commit 96290ad into zlib-ng:develop Sep 3, 2021
Dead2 added a commit that referenced this pull request Dec 13, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1035 #1051 #1056 #1063 #1067
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050
@Dead2 Dead2 mentioned this pull request Dec 13, 2021
Dead2 added a commit that referenced this pull request Dec 20, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix deflateBound and compressBound returning too small size estimates #1071
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1032 #1035 #1051 #1056 #1063 #1067
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050 #1075
Dead2 added a commit that referenced this pull request Dec 20, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix deflateBound and compressBound returning too small size estimates #1071
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1032 #1035 #1051 #1056 #1063 #1067
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050 #1075
Dead2 added a commit that referenced this pull request Dec 20, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix deflateBound and compressBound returning too small size estimates #1071
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1032 #1035 #1051 #1056 #1063 #1067
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050 #1075
Dead2 added a commit that referenced this pull request Dec 20, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix deflateBound and compressBound returning too small size estimates #1049 #1071
 - Fix incorrect function declaration warning #1080
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1032 #1035 #1049 #1051 #1056 #1063 #1067
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050 #1075
Dead2 added a commit that referenced this pull request Dec 20, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix deflateBound and compressBound returning too small size estimates #1049 #1071
 - Fix incorrect function declaration warning #1080
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1032 #1035 #1049 #1051 #1056 #1063 #1067 #1079
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050 #1075
Dead2 added a commit that referenced this pull request Dec 24, 2021
 - Fix hangs on macOS #1031
 - Fix minideflate write buffers being overwritten #1060
 - Fix deflateBound and compressBound returning too small size estimates #1049 #1071
 - Fix incorrect function declaration warning #1080
 - Fix build problems when building outside of source dir #1049
 - Fix build problems on arm2-7 #1030
 - Fixed some compile warnings #1020 #1036 #1037 #1048
 - Improved posix memalign support #888
 - Improvements to testing #637 #1026 #1032 #1035 #1049 #1051 #1056 #1063 #1067 #1079
 - Improvements for integration into other projects #1022 #1042
 - Code style fixes #637 #1040 #1050 #1075
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Improving maintainability or removing code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants