diff options
| author | Jos Collin <jcollin@redhat.com> | 2017-04-22 10:19:20 +0200 |
|---|---|---|
| committer | Jos Collin <jcollin@redhat.com> | 2017-04-24 05:35:28 +0200 |
| commit | 59fb7ae1543b5ff1d39b45ecc2b4423293fe9e32 (patch) | |
| tree | 31bae10e264b4cb10fff3041f63b447f6f7bb3fa /src/osdc | |
| parent | Merge pull request #14707 from wjwithagen/wip-wjw-freebsd-kqueue-logging (diff) | |
| download | ceph-59fb7ae1543b5ff1d39b45ecc2b4423293fe9e32.tar.xz ceph-59fb7ae1543b5ff1d39b45ecc2b4423293fe9e32.zip | |
osdc: warning: comparison between signed and unsigned integer expressions
The following warning appears during make. Fixed the review comments too.
[ 28%] Building CXX object src/mds/CMakeFiles/mds.dir/__/osdc/Journaler.cc.o
./src/osdc/Journaler.cc: In member function ‘void Journaler::_do_flush(unsigned int)’:
./src/osdc/Journaler.cc:643:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (newlen < len) {
~~~~~~~^~~~~
Signed-off-by: Jos Collin <jcollin@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | src/osdc/Journaler.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/osdc/Journaler.cc b/src/osdc/Journaler.cc index 642b15be430..7c1be262644 100644 --- a/src/osdc/Journaler.cc +++ b/src/osdc/Journaler.cc @@ -640,7 +640,7 @@ void Journaler::_do_flush(unsigned amount) waiting_for_zero = true; return; } - if (newlen < len) { + if (static_cast<uint64_t>(newlen) < len) { ldout(cct, 10) << "_do_flush wanted to do " << flush_pos << "~" << len << " but hit prezero_pos " << prezero_pos << ", will do " << flush_pos << "~" << newlen << dendl; |