From 163e0158dc5bc08bf1f43f209ec80ed65ca1b427 Mon Sep 17 00:00:00 2001
From: Kaveh Vahedipour <kaveh@arangodb.com>
Date: Fri, 20 Jan 2017 15:22:30 +0100
Subject: [PATCH] before cppcheck enthusiasts start slacking :)

---
 arangod/Agency/Supervision.cpp | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/arangod/Agency/Supervision.cpp b/arangod/Agency/Supervision.cpp
index e4b4d916ab..4c095807be 100644
--- a/arangod/Agency/Supervision.cpp
+++ b/arangod/Agency/Supervision.cpp
@@ -110,8 +110,6 @@ std::vector<check_t> Supervision::checkDBServers() {
   Node::Children const serversRegistered =
       _snapshot(currentServersRegisteredPrefix).children();
 
-  bool reportPersistent = false;
-
   std::vector<std::string> todelete;
   for (auto const& machine : _snapshot(healthPrefix).children()) {
     if (machine.first.substr(0, 2) == "DB") {
@@ -121,6 +119,7 @@ std::vector<check_t> Supervision::checkDBServers() {
 
   for (auto const& machine : machinesPlanned) {
     bool good = false;
+    bool reportPersistent = false;
     std::string lastHeartbeatTime, lastHeartbeatAcked, lastStatus,
         heartbeatTime, heartbeatStatus, serverID;
 
@@ -257,8 +256,6 @@ std::vector<check_t> Supervision::checkDBServers() {
 // Check all coordinators, guarded above doChecks
 std::vector<check_t> Supervision::checkCoordinators() {
 
-  bool reportPersistent = false;
-
   std::vector<check_t> ret;
   Node::Children const& machinesPlanned =
       _snapshot(planCoordinatorsPrefix).children();
@@ -280,6 +277,7 @@ std::vector<check_t> Supervision::checkCoordinators() {
   }
 
   for (auto const& machine : machinesPlanned) {
+    bool reportPersistent = false;
     bool good = false;
     std::string lastHeartbeatTime, lastHeartbeatAcked, lastStatus,
         heartbeatTime, heartbeatStatus, serverID;