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: drop using std ns in header files and other cleanups #15137

Merged
merged 2 commits into from May 18, 2017

Conversation

Projects
None yet
3 participants
@theanalyst
Member

theanalyst commented May 17, 2017

We're still having the include in include/type.h, so just sweeping things
under the rug for now, also dropped the expat include in a couple of
header files

theanalyst added some commits May 17, 2017

rgw: drop using namespace std in header files
We're still having the include in include/type.h, so just sweeping
things under the rug for now

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
rgw: drop expat header in a couple of files
Since it is already a part of rgw_xml
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

@theanalyst theanalyst requested review from tchaikov and cbodley May 17, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 17, 2017

might need to push it to ceph-ci to make sure it builds on more platforms.

@theanalyst

This comment has been minimized.

@tchaikov tchaikov merged commit b4648bf into ceph:master May 18, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment