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

pool: fix coverity issue 374612 #5866

Merged
merged 1 commit into from
Aug 21, 2023
Merged

Conversation

grom72
Copy link
Contributor

@grom72 grom72 commented Aug 21, 2023

This change is Reviewable

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
@grom72 grom72 added pmempool src/libpmempool and src/tools/pmempool no changelog Add to skip the changelog check on your pull request labels Aug 21, 2023
@grom72 grom72 added this to the 2.0.1 milestone Aug 21, 2023
@grom72 grom72 requested a review from janekmi August 21, 2023 09:22
Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @grom72)

@codecov
Copy link

codecov bot commented Aug 21, 2023

Codecov Report

Merging #5866 (9f44199) into master (0de65de) will increase coverage by 1.69%.
The diff coverage is 0.00%.

❗ Current head 9f44199 differs from pull request most recent head 49b17a4. Consider uploading reports for the commit 49b17a4 to get more accurate results

@@            Coverage Diff             @@
##           master    #5866      +/-   ##
==========================================
+ Coverage   68.68%   70.37%   +1.69%     
==========================================
  Files         131      131              
  Lines       19682    19180     -502     
  Branches     3259     3193      -66     
==========================================
- Hits        13518    13498      -20     
+ Misses       6164     5682     -482     

@janekmi janekmi merged commit 479a965 into pmem:master Aug 21, 2023
9 of 10 checks passed
@grom72 grom72 deleted the fix-coverity-374612 branch October 24, 2023 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Add to skip the changelog check on your pull request pmempool src/libpmempool and src/tools/pmempool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants