Remove unnecessary sai header include from OrchBase and use const for…

… reference Signed-off-by: Jipan Yang <[email protected]>
sonic-net · Oct 25, 2017 · 34b36f6 · 34b36f6
1 parent f567979
commit 34b36f6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/orchagent/orchbase.cpp b/orchagent/orchbase.cpp
@@ -16,7 +16,7 @@ OrchBase::OrchBase(DBConnector *db, string tableName)
     addConsumer(db, tableName);
 }
 
-OrchBase::OrchBase(DBConnector *db, vector<string> &tableNames)
+OrchBase::OrchBase(DBConnector *db, const vector<string> &tableNames)
 {
     for(auto it : tableNames)
     {
@@ -199,4 +199,4 @@ void OrchBase::addConsumer(DBConnector *db, string tableName, int batchSize)
         Consumer consumer(new ConsumerStateTable(db, tableName, batchSize));
         m_consumerMap.insert(ConsumerMapPair(tableName, consumer));
     }
-}
+}
diff --git a/orchagent/orchbase.h b/orchagent/orchbase.h
@@ -4,11 +4,6 @@
 #include <map>
 #include <memory>
 
-extern "C" {
-#include "sai.h"
-#include "saistatus.h"
-}
-
 #include "dbconnector.h"
 #include "table.h"
 #include "consumertable.h"
@@ -38,7 +33,7 @@ class OrchBase
 {
 public:
     OrchBase(DBConnector *db, string tableName);
-    OrchBase(DBConnector *db, vector<string> &tableNames);
+    OrchBase(DBConnector *db, const vector<string> &tableNames);
     OrchBase(const vector<TableConnector>& tables);
     virtual ~OrchBase();
 
@@ -60,4 +55,4 @@ class OrchBase
     bool syncDB(string tableName, Table &tableConsumer);
 };
 
-#endif /* SWSS_ORCH_BASE_H */
+#endif /* SWSS_ORCH_BASE_H */