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

os/bluestore: silence gcc warning #14028

Merged
merged 1 commit into from Mar 20, 2017

Conversation

Projects
None yet
2 participants
@tchaikov
Contributor

tchaikov commented Mar 18, 2017

src/os/bluestore/BitAllocator.h:410:8: warning: ‘virtual bool
BitMapAreaIN::child_check_n_lock(BitMapArea*, int64_t, bool)’ was
hidden [-Woverloaded-virtual]
bool child_check_n_lock(BitMapArea child, int64_t required, bool
lock) {
^~~~~~~~~~~~~~~~~~
/var/ceph/ceph/src/os/bluestore/BitAllocator.h:489:8: warning: by
‘BitMapAreaLeaf::child_check_n_lock’ [-Woverloaded-virtual]
bool child_check_n_lock(BitMapZone
child, int64_t required, bool
lock);
^~~~~~~~~~~~~~~~~~

Signed-off-by: Kefu Chai kchai@redhat.com

os/bluestore: silence gcc warning
src/os/bluestore/BitAllocator.h:410:8: warning: ‘virtual bool
BitMapAreaIN::child_check_n_lock(BitMapArea*, int64_t, bool)’ was
hidden    [-Woverloaded-virtual]
 bool child_check_n_lock(BitMapArea *child, int64_t required, bool
lock) {
      ^~~~~~~~~~~~~~~~~~
 /var/ceph/ceph/src/os/bluestore/BitAllocator.h:489:8: warning:   by
 ‘BitMapAreaLeaf::child_check_n_lock’ [-Woverloaded-virtual]
  bool child_check_n_lock(BitMapZone* child, int64_t required, bool
  lock);
       ^~~~~~~~~~~~~~~~~~

Signed-off-by: Kefu Chai <kchai@redhat.com>
@badone

badone approved these changes Mar 18, 2017

LGTM

@tchaikov tchaikov merged commit a46e6c9 into ceph:master Mar 20, 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

@tchaikov tchaikov deleted the tchaikov:wip-kill-gcc-warning branch Mar 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment