Remove extra semicolons (#1331) (#1345)

Author: ninadpahunems

lib/api/AuthTokensController.cpp

@@ -9,7 +9,7 @@
 
 namespace MAT_NS_BEGIN {
 
-    MATSDK_LOG_INST_COMPONENT_CLASS(AuthTokensController, "EventsSDK.AuthTokensController", "Events telemetry client - AuthTokensController class");
+    MATSDK_LOG_INST_COMPONENT_CLASS(AuthTokensController, "EventsSDK.AuthTokensController", "Events telemetry client - AuthTokensController class")
 
     AuthTokensController::AuthTokensController()
         :m_IsStrictModeEnabled(false)

lib/api/DataViewerCollection.cpp

@@ -8,7 +8,7 @@
 
 namespace MAT_NS_BEGIN {
 
-    MATSDK_LOG_INST_COMPONENT_CLASS(DataViewerCollection, "EventsSDK.DataViewerCollection", "Microsoft Telemetry Client - DataViewerCollection class");
+    MATSDK_LOG_INST_COMPONENT_CLASS(DataViewerCollection, "EventsSDK.DataViewerCollection", "Microsoft Telemetry Client - DataViewerCollection class")
 
     void DataViewerCollection::DispatchDataViewerEvent(const std::vector<uint8_t>& packetData) const noexcept
     {
@@ -21,7 +21,7 @@ namespace MAT_NS_BEGIN {
             // Task 3568800: Integrate ThreadPool to IDataViewerCollection
             viewer->ReceiveData(packetData);
         }
-    };
+    }
 
     void DataViewerCollection::RegisterViewer(const std::shared_ptr<IDataViewer>& dataViewer)
     {

lib/api/DataViewerCollection.hpp

@@ -31,7 +31,7 @@ namespace MAT_NS_BEGIN {
 
         virtual bool IsViewerRegistered(const char* viewerName) const override;
 
-        virtual ~DataViewerCollection() noexcept {};
+        virtual ~DataViewerCollection() noexcept {}
     private:
         MATSDK_LOG_DECL_COMPONENT_CLASS();
 

lib/api/IRuntimeConfig.hpp

@@ -209,7 +209,7 @@ namespace MAT_NS_BEGIN
         /// </summary>
         virtual bool SkipIKeyRegistration() const = 0;
 
-        virtual ~IRuntimeConfig() {};
+        virtual ~IRuntimeConfig() {}
     };
 
     /// @endcond

lib/api/LogManagerImpl.cpp

@@ -128,7 +128,7 @@ namespace MAT_NS_BEGIN
         return true;
     }
 
-    MATSDK_LOG_INST_COMPONENT_CLASS(LogManagerImpl, "EventsSDK.LogManager", "Microsoft Telemetry Client - LogManager class");
+    MATSDK_LOG_INST_COMPONENT_CLASS(LogManagerImpl, "EventsSDK.LogManager", "Microsoft Telemetry Client - LogManager class")
 
 #if 1
     // TODO: integrate Tracing API from v1
@@ -374,7 +374,7 @@ namespace MAT_NS_BEGIN
             client->SetMsRootCheck(m_logConfiguration[CFG_MAP_HTTP][CFG_BOOL_HTTP_MS_ROOT_CHECK]);
         }
 #endif
-    };
+    }
 
     LogManagerImpl::~LogManagerImpl() noexcept
     {
@@ -541,7 +541,7 @@ namespace MAT_NS_BEGIN
     const std::string& LogManagerImpl::GetTransmitProfileName()
     {
         return TransmitProfiles::getProfile();
-    };
+    }
 
     ISemanticContext& LogManagerImpl::GetSemanticContext()
     {
@@ -699,7 +699,7 @@ namespace MAT_NS_BEGIN
     void LogManagerImpl::AddEventListener(DebugEventType type, DebugEventListener& listener)
     {
         m_debugEventSource.AddEventListener(type, listener);
-    };
+    }
 
     /// <summary>
     /// Removes the event listener.
@@ -709,7 +709,7 @@ namespace MAT_NS_BEGIN
     void LogManagerImpl::RemoveEventListener(DebugEventType type, DebugEventListener& listener)
     {
         m_debugEventSource.RemoveEventListener(type, listener);
-    };
+    }
 
     /// <summary>
     /// Dispatches the event.
@@ -719,7 +719,7 @@ namespace MAT_NS_BEGIN
     bool LogManagerImpl::DispatchEvent(DebugEvent evt)
     {
         return m_debugEventSource.DispatchEvent(std::move(evt));
-    };
+    }
 
     /// <summary>Attach cascaded DebugEventSource to forward all events to</summary>
     bool LogManagerImpl::AttachEventSource(DebugEventSource& other)

lib/api/LogManagerImpl.hpp

@@ -182,7 +182,7 @@ namespace MAT_NS_BEGIN
         {
             const std::string val(value);
             return SetContext(name, val, piiKind);
-        };
+        }
 
         virtual inline status_t SetContext(const std::string& name, int8_t value, PiiKind piiKind = PiiKind_None) override
         {

lib/api/Logger.cpp

@@ -175,37 +175,37 @@ namespace MAT_NS_BEGIN
     void Logger::SetContext(const std::string& k, const char v[], PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     void Logger::SetContext(const std::string& k, const std::string& v, PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     void Logger::SetContext(const std::string& k, double v, PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     void Logger::SetContext(const std::string& k, int64_t v, PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     void Logger::SetContext(const std::string& k, time_ticks_t v, PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     void Logger::SetContext(const std::string& k, GUID_t v, PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     void Logger::SetContext(const std::string& k, bool v, PiiKind pii)
     {
         SetContext(k, EventProperty(v, pii));
-    };
+    }
 
     // The goal of this method is to rewire the logger instance to any other ISemanticContext issued by SDK.
     // SDK may provide a future option for a guest logger to opt-in into its own semantic context. The method will then

lib/config/RuntimeConfig_Default.hpp

@@ -115,14 +115,14 @@ namespace MAT_NS_BEGIN
             UNREFERENCED_PARAMETER(extension);
             UNREFERENCED_PARAMETER(experimentationProject);
             UNREFERENCED_PARAMETER(eventName);
-        };
+        }
 
         virtual EventLatency GetEventLatency(std::string const& tenantId = std::string(), std::string const& eventName = std::string()) override
         {
             UNREFERENCED_PARAMETER(tenantId);
             UNREFERENCED_PARAMETER(eventName);
             return EventLatency_Normal;
-        };
+        }
 
         virtual std::string GetMetaStatsTenantToken() override
         {
@@ -134,7 +134,7 @@ namespace MAT_NS_BEGIN
                     return std::string(token);
             }
             return std::string(defaultToken);
-        };
+        }
 
         virtual unsigned GetMetaStatsSendIntervalSec() override
         {
@@ -144,7 +144,7 @@ namespace MAT_NS_BEGIN
         virtual unsigned GetOfflineStorageMaximumSizeBytes() override
         {
             return config[CFG_INT_CACHE_FILE_SIZE];
-        };
+        }
 
         virtual unsigned GetOfflineStorageResizeThresholdPct() override
         {

lib/decorators/BaseDecorator.hpp

@@ -24,7 +24,7 @@ namespace MAT_NS_BEGIN
 
     public:
         BaseDecorator(ILogManager& owner);
-        virtual ~BaseDecorator() {};
+        virtual ~BaseDecorator() {}
         bool decorate(CsProtocol::Record& record);
 
     protected:

lib/decorators/EventPropertiesDecorator.hpp

@@ -53,7 +53,7 @@ namespace MAT_NS_BEGIN {
             //
             randomLocalId = "r:";
             randomLocalId+= PAL::generateUuidString();
-        };
+        }
 
         void dropPiiPartA(::CsProtocol::Record& record)
         {