Clean code, remove unused header file (sonic-net#1468)

theasianpianist · Oct 22, 2020 · 4de6db0 · 4de6db0
1 parent bb94f8e
commit 4de6db0
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 8 deletions.
diff --git a/mclagsyncd/mclaglink.h b/mclagsyncd/mclaglink.h
@@ -36,7 +36,6 @@
 
 #include "producerstatetable.h"
 #include "selectable.h"
-#include "redisclient.h"
 #include "mclagsyncd/mclag.h"
 
 namespace swss {

diff --git a/orchagent/bufferorch.h b/orchagent/bufferorch.h
@@ -7,7 +7,6 @@
 #include "orch.h"
 #include "portsorch.h"
 #include "redisapi.h"
-#include "redisclient.h"
 
 #define BUFFER_POOL_WATERMARK_STAT_COUNTER_FLEX_COUNTER_GROUP "BUFFER_POOL_WATERMARK_STAT_COUNTER"
 

diff --git a/orchagent/countercheckorch.cpp b/orchagent/countercheckorch.cpp
@@ -2,7 +2,6 @@
 #include "portsorch.h"
 #include "select.h"
 #include "notifier.h"
-#include "redisclient.h"
 #include "sai_serialize.h"
 #include <inttypes.h>
 

diff --git a/orchagent/flexcounterorch.cpp b/orchagent/flexcounterorch.cpp
@@ -3,7 +3,6 @@
 #include "portsorch.h"
 #include "select.h"
 #include "notifier.h"
-#include "redisclient.h"
 #include "sai_serialize.h"
 #include "pfcwdorch.h"
 #include "bufferorch.h"

diff --git a/orchagent/pfcwdorch.cpp b/orchagent/pfcwdorch.cpp
@@ -8,7 +8,6 @@
 #include "redisapi.h"
 #include "select.h"
 #include "notifier.h"
-#include "redisclient.h"
 #include "schema.h"
 #include "subscriberstatetable.h"
 

diff --git a/orchagent/pfcwdorch.h b/orchagent/pfcwdorch.h
@@ -7,7 +7,6 @@
 #include "producertable.h"
 #include "notificationconsumer.h"
 #include "timer.h"
-#include "redisclient.h"
 
 extern "C" {
 #include "sai.h"

diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -25,7 +25,6 @@
 #include "crmorch.h"
 #include "countercheckorch.h"
 #include "notifier.h"
-#include "redisclient.h"
 #include "fdborch.h"
 
 extern sai_switch_api_t *sai_switch_api;

diff --git a/orchagent/routeorch.cpp b/orchagent/routeorch.cpp
@@ -596,7 +596,7 @@ void RouteOrch::doTask(Consumer& consumer)
                     else
                     {
                         if (addRoute(ctx, nhg))
-                             it = consumer.m_toSync.erase(it);
+                            it = consumer.m_toSync.erase(it);
                         else
                             it++;
                     }