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

common/Finisher: fix uninitialized variable warning #14958

Merged
merged 1 commit into from May 5, 2017

Conversation

Projects
None yet
3 participants
@branch-predictor
Member

branch-predictor commented May 4, 2017

This silences the following warning:

Finisher.cc: In member function ‘void*
Finisher::finisher_thread_entry()’:
Finisher.cc:89:42: warning: ‘count’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
logger->dec(l_finisher_queue_len, count);
^

Signed-off-by: Piotr Dałek piotr.dalek@corp.ovh.com

Finisher: fix uninitialized variable warning
This silences the following warning:

Finisher.cc: In member function ‘void*
Finisher::finisher_thread_entry()’:
Finisher.cc:89:42: warning: ‘count’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
  logger->dec(l_finisher_queue_len, count);
                                         ^

Signed-off-by: Piotr Dałek <piotr.dalek@corp.ovh.com>

@tchaikov tchaikov added the cleanup label May 4, 2017

@liewegas liewegas changed the title from Finisher: fix uninitialized variable warning to common/Finisher: fix uninitialized variable warning May 4, 2017

@liewegas liewegas added the needs-qa label May 4, 2017

@liewegas liewegas merged commit 209a4b8 into ceph:master May 5, 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

@branch-predictor branch-predictor deleted the ovh:bp-fix-finisher-warning branch May 23, 2017

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