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: add override in header files mostly #13586
Conversation
src/rgw/rgw_rest.h
Outdated
virtual void init(RGWRados *store, struct req_state *s, | ||
RGWHandler *dialect_handler) { | ||
void init(RGWRados *store, struct req_state *s, | ||
RGWHandler *dialect_handler) override { |
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.
indentation
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.
done
src/rgw/rgw_rest_s3.h
Outdated
struct req_state *s, | ||
rgw::io::BasicClient *cio); | ||
virtual int authorize() { | ||
rgw::io::BasicClient *cio) override; |
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.
indentation
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.
done
src/rgw/rgw_rest_s3.h
Outdated
struct req_state *s, | ||
rgw::io::BasicClient *cio); | ||
virtual int authorize() { | ||
rgw::io::BasicClient *cio) override; |
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.
indentation
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.
done
src/rgw/rgw_swift_auth.h
Outdated
|
||
virtual RGWRESTMgr *get_resource_mgr(struct req_state* const s, | ||
RGWRESTMgr *get_resource_mgr(struct req_state* const s, |
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.
indentation
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.
done
src/rgw/rgw_swift_auth.h
Outdated
const std::string& uri, | ||
std::string* const out_uri) override { | ||
return this; | ||
} | ||
|
||
virtual RGWHandler_REST* get_handler(struct req_state*, | ||
RGWHandler_REST* get_handler(struct req_state*, |
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.
indentation
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.
done
@cbodley It seems that all issues have been cleaned. |
@cbodley hi, could we merge it now? |
@liuchang0812 just trying to be careful about the timing for merge, because we'll need to rebase a lot of other PRs afterwards. what do you think, @yehudasa? is there stuff you want to get merged before this? |
@cbodley okay |
@cbodley @liuchang0812 yeah, let's get wip-bucket-cleanup in first. |
@liuchang0812 can you please rebase? |
copy that, I will rebase it soon
2017-03-14 3:23 GMT+08:00 Casey Bodley <notifications@github.com>:
… @liuchang0812 <https://github.com/Liuchang0812> can you please rebase?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#13586 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABtHdtBAhtUqE5ZAUUEGPXqdsb6uVcQUks5rlZeYgaJpZM4MIlNo>
.
|
110f240
to
90ba70e
Compare
change log:
|
Signed-off-by: liuchang0812 <liuchang0812@gmail.com>
90ba70e
to
3b24747
Compare
Signed-off-by: liuchang0812 liuchang0812@gmail.com