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/PG: fix warning #16655

Merged
merged 1 commit into from Jul 28, 2017

Conversation

Projects
None yet
2 participants
@liewegas
Member

liewegas commented Jul 28, 2017

Introduced by 4af49a8

Signed-off-by: Sage Weil sage@redhat.com

osd/PG: fix warning
Introduced by 4af49a8

Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas requested a review from gregsfortytwo Jul 28, 2017

@liewegas liewegas added this to the luminous milestone Jul 28, 2017

@liewegas

This comment has been minimized.

Member

liewegas commented Jul 28, 2017

oops

@liewegas

This comment has been minimized.

Member

liewegas commented Jul 28, 2017

A reminder that the "warnings are errors" thing never actually landed (or doesn't work as expected, at least not on my compiler version, gcc (GCC) 6.2.1 20160916 (Red Hat 6.2.1-2)).

@gregsfortytwo

This comment has been minimized.

Member

gregsfortytwo commented Jul 28, 2017

Whoops.

And yeah, I don't think it ever landed.

@gregsfortytwo

Reviewed-by: Greg Farnum gfarnum@redhat.com

@gregsfortytwo gregsfortytwo merged commit dce8e7d into ceph:master Jul 28, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment