]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
test/msgr: silence warnings from -Wsign-compare 15356/head
authorJos Collin <jcollin@redhat.com>
Tue, 30 May 2017 05:19:15 +0000 (10:49 +0530)
committerJos Collin <jcollin@redhat.com>
Tue, 30 May 2017 05:19:23 +0000 (10:49 +0530)
The following warning appears during make:

ceph/src/test/msgr/test_msgr.cc: In member function ‘virtual void MessengerTest_MarkdownTest_Test::TestBody()’:
ceph/src/test/msgr/test_msgr.cc:1474:46: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     CHECK_AND_WAIT_TRUE(srv_dispatcher.count > last + 1);
                         ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~
ceph/src/test/msgr/test_msgr.cc:54:9: note: in definition of macro ‘CHECK_AND_WAIT_TRUE’
     if (expr)                           \
         ^~~~
ceph/src/test/msgr/test_msgr.cc:1475:30: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (srv_dispatcher.count == last) {
         ~~~~~~~~~~~~~~~~~~~~~^~~~~~~

Signed-off-by: Jos Collin <jcollin@redhat.com>
src/test/msgr/test_msgr.cc

index 831f71aa85e54267edb8a1586f1f60b044420d9c..948cdeebdfbbf961d3a21832f333b563a6697651 100644 (file)
@@ -1461,7 +1461,7 @@ TEST_P(MessengerTest, MarkdownTest) {
   client_msgr->start();
 
   int i = 1000;
-  uint64_t last = 0;
+  int64_t last = 0;
   bool equal = false;
   uint64_t equal_count = 0;
   while (i--) {