8000 Fix the build on less common platforms in zmalloc.c by natoscott · Pull Request #1922 · valkey-io/valkey · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix the build on less common platforms in zmalloc.c #1922

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

8000
Merged
merged 1 commit into from
Apr 7, 2025

Conversation

natoscott
Copy link
Contributor

Corrects a mismatched type declaration for the used_memory_thread pointer into the used_memory_thread_padded array (global scope) - in the less used else branch of a cpp platform conditional.

Fixes #1916

Corrects a mismatched type declaration for the used_memory_thread
pointer into the used_memory_thread_padded array (global scope) -
in the less used else branch of a cpp platform conditional.

Fixes valkey-io#1916

Signed-off-by: Nathan Scott <nathans@redhat.com>
Copy link
codecov bot commented Apr 7, 2025
8000

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.05%. Comparing base (dfdcbfe) to head (72909ac).
Report is 1 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable    #1922      +/-   ##
============================================
+ Coverage     71.02%   71.05%   +0.02%     
============================================
  Files           123      123              
  Lines         65683    65683              
============================================
+ Hits          46653    46668      +15     
+ Misses        19030    19015      -15     
Files with missing lines Coverage Δ
src/zmalloc.c 82.74% <ø> (ø)

... and 13 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@enjoy-binbin enjoy-binbin requested a review from zuiderkwast April 7, 2025 11:20
@zuiderkwast zuiderkwast moved this to To be backported in Valkey 8.0 Apr 7, 2025
@zuiderkwast zuiderkwast removed this from Valkey 8.0 Apr 7, 2025
@zuiderkwast zuiderkwast moved this to To be backported in Valkey 8.1 Apr 7, 2025
@zuiderkwast zuiderkwast merged commit 204097d into valkey-io:unstable Apr 7, 2025
51 checks passed
@natoscott natoscott deleted the fix-build-s390x-ppc64le branch April 7, 2025 22:57
murphyjacob4 pushed a commit to enjoy-binbin/valkey that referenced this pull request Apr 13, 2025
Corrects a mismatched type declaration for the used_memory_thread
pointer into the used_memory_thread_padded array (global scope) - in the
less used else branch of a cpp platform conditional.

Fixes valkey-io#1916

Signed-off-by: Nathan Scott <nathans@redhat.com>
murphyjacob4 pushed a commit to murphyjacob4/valkey that referenced this pull request Apr 18, 2025
Corrects a mismatched type declaration for the used_memory_thread
pointer into the used_memory_thread_padded array (global scope) - in the
less used else branch of a cpp platform conditional.

Fixes valkey-io#1916

Signed-off-by: Nathan Scott <nathans@redhat.com>
murphyjacob4 pushed a commit to murphyjacob4/valkey that referenced this pull request Apr 18, 2025
Corrects a mismatched type declaration for the used_memory_thread
pointer into the used_memory_thread_padded array (global scope) - in the
less used else branch of a cpp platform conditional.

Fixes valkey-io#1916

Signed-off-by: Nathan Scott <nathans@redhat.com>
Signed-off-by: Jacob Murphy <jkmurphy@google.com>
@zuiderkwast zuiderkwast moved this from To be backported to 8.1.1 in Valkey 8.1 Apr 22, 2025
murphyjacob4 pushed a commit to murphyjacob4/valkey that referenced this pull request Apr 23, 2025
Corrects a mismatched type declaration for the used_memory_thread
pointer into the used_memory_thread_padded array (global scope) - in the
less used else branch of a cpp platform conditional.

Fixes valkey-io#1916

Signed-off-by: Nathan Scott <nathans@redhat.com>
Signed-off-by: Jacob Murphy <jkmurphy@google.com>
zuiderkwast pushed a commit that referenced this pull request Apr 23, 2025
Corrects a mismatched type declaration for the used_memory_thread
pointer into the used_memory_thread_padded array (global scope) - in the
less used else branch of a cpp platform conditional.

Fixes #1916

Signed-off-by: Nathan Scott <nathans@redhat.com>
Signed-off-by: Jacob Murphy <jkmurphy@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 8.1.1
Development

Successfully merging this pull request may close these issues.

[BUG] build failure for 8.1.0 on s390x and ppc64le platforms
3 participants
0