[syncd] Fix SwitchNotifications to be const reference (#945)

To satisfy LGTM alert and for performance reasons to skip making unnecessary copy of struct.
sonic-net · Sep 25, 2021 · 5863ed3 · 5863ed3
1 parent d598dea
commit 5863ed3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/syncd/SwitchNotifications.cpp b/syncd/SwitchNotifications.cpp
@@ -5,7 +5,7 @@
 using namespace syncd;
 
 SwitchNotifications::SlotBase::SlotBase(
-        _In_ sai_switch_notifications_t sn):
+        _In_ const sai_switch_notifications_t& sn):
     m_handler(nullptr),
     m_sn(sn)
 {

diff --git a/syncd/SwitchNotifications.h b/syncd/SwitchNotifications.h
@@ -20,7 +20,7 @@ namespace syncd
                 public:
 
                     SlotBase(
-                            _In_ sai_switch_notifications_t sn);
+                            _In_ const sai_switch_notifications_t& sn);
 
                     virtual ~SlotBase();