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
osd/PrimaryLogPG: drop unused parameters #18581
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See, that function is used here:
/home/jenkins-build/build/workspace/ceph-pull-requests/src/osd/PrimaryLogPG.cc:7026:43: error: no matching function for call to ‘PrimaryLogPG::setattr_maybe_cache(ObjectContextRef&, PrimaryLogPG::OpContext*&, PGTransaction*&, const char [2], ceph::bufferlist&)’
setattr_maybe_cache(obc, ctx, t, OI_ATTR, bv);
35c5176
to
282a41d
Compare
@joscollin , thank you for your notice, it has been changed now, please check it out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wzliao As you are doing a cleanup, What about the other unused parameters in those function definitions ?
@joscollin , i put on the '-Wunused-parameter' option when build the project, got thousands '-Wunused-parameter' warnings, some of them should remain what they are for capability, some of them can be changed, but can not be easily found out, i think we should let they be. |
src/osd/PrimaryLogPG.cc
Outdated
@@ -12536,7 +12536,7 @@ void PrimaryLogPG::hit_set_persist() | |||
map <string, bufferlist> attrs; | |||
attrs[OI_ATTR].claim(boi); | |||
attrs[SS_ATTR].claim(bss); | |||
setattrs_maybe_cache(ctx->obc, ctx.get(), ctx->op_t.get(), attrs); | |||
setattrs_maybe_cache(ctx->obc, ctx->op_t.get(), attrs); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Fix the unwanted space after ctx->obc,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joscollin , fixed it
282a41d
to
8ed2457
Compare
Signed-off-by: Liao Weizhong <liaoweizhong@unionpay.com>
osd/PrimaryLogPG: drop unused parameters