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
rgw: Remove unused Parameter in Function RGWConf::init() #17129
Conversation
Signed-off-by: Wen Zhang <zhangwen1@unionpay.com>
Jenkins Retest please |
@xiexingguo Could you review it? Tks~ |
I am forwarding this to @joscollin |
Jenkins retest this please |
Jenkins retest this please |
@@ -385,7 +385,7 @@ class RGWEnv; | |||
class RGWConf { | |||
friend class RGWEnv; | |||
protected: |
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.
Drop protected
as it is not being used and RGWEnv
is a friend of RGWConf
.
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.
protected has been dropped in patch #17432
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.
@C2python Yes. As your response got delayed, I have submitted that in my patch. I will continue the review of this. Thanks.
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.
LGTM
Signed-off-by: Wen Zhang zhangwen1@unionpay.com