]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: Remove redundant local variable declaration 19812/head
authorShinobu Kinjo <shinobu@redhat.com>
Sat, 6 Jan 2018 02:06:33 +0000 (11:06 +0900)
committerShinobu Kinjo <shinobu@redhat.com>
Sat, 6 Jan 2018 02:06:33 +0000 (11:06 +0900)
Signed-off-by: Shinobu Kinjo <shinobu@redhat.com>
src/osd/OSD.cc

index 4633f97b3d310a1c2e69a4e40bf248c5ef51bb5e..b19c48a63c2af6c53504773a5e87ef28869fc759 100644 (file)
@@ -4595,7 +4595,7 @@ void OSD::maybe_update_heartbeat_peers()
 
   // include next and previous up osds to ensure we have a fully-connected set
   set<int> want, extras;
-  int next = osdmap->get_next_up_osd_after(whoami);
+  const int next = osdmap->get_next_up_osd_after(whoami);
   if (next >= 0)
     want.insert(next);
   int prev = osdmap->get_previous_up_osd_before(whoami);
@@ -4624,8 +4624,7 @@ void OSD::maybe_update_heartbeat_peers()
   }
 
   // too few?
-  int start = osdmap->get_next_up_osd_after(whoami);
-  for (int n = start; n >= 0; ) {
+  for (int n = next; n >= 0; ) {
     if ((int)heartbeat_peers.size() >= cct->_conf->osd_heartbeat_min_peers)
       break;
     if (!extras.count(n) && !want.count(n) && n != whoami) {
@@ -4634,7 +4633,7 @@ void OSD::maybe_update_heartbeat_peers()
       _add_heartbeat_peer(n);
     }
     n = osdmap->get_next_up_osd_after(n);
-    if (n == start)
+    if (n == next)
       break;  // came full circle; stop
   }