Skip to content

Commit bb3dc6c

Browse files
chrdavisChris Davis
chrdavis
and
Chris Davis
authored
Remove extra semicolons (#1331)
Co-authored-by: Chris Davis <[email protected]>
1 parent d4d947d commit bb3dc6c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

48 files changed

+220
-220
lines changed

‎lib/api/AuthTokensController.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
namespace MAT_NS_BEGIN {
1111

12-
MATSDK_LOG_INST_COMPONENT_CLASS(AuthTokensController, "EventsSDK.AuthTokensController", "Events telemetry client - AuthTokensController class");
12+
MATSDK_LOG_INST_COMPONENT_CLASS(AuthTokensController, "EventsSDK.AuthTokensController", "Events telemetry client - AuthTokensController class")
1313

1414
AuthTokensController::AuthTokensController()
1515
:m_IsStrictModeEnabled(false)

‎lib/api/DataViewerCollection.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
namespace MAT_NS_BEGIN {
1010

11-
MATSDK_LOG_INST_COMPONENT_CLASS(DataViewerCollection, "EventsSDK.DataViewerCollection", "Microsoft Telemetry Client - DataViewerCollection class");
11+
MATSDK_LOG_INST_COMPONENT_CLASS(DataViewerCollection, "EventsSDK.DataViewerCollection", "Microsoft Telemetry Client - DataViewerCollection class")
1212

1313
void DataViewerCollection::DispatchDataViewerEvent(const std::vector<uint8_t>& packetData) const noexcept
1414
{
@@ -21,7 +21,7 @@ namespace MAT_NS_BEGIN {
2121
// Task 3568800: Integrate ThreadPool to IDataViewerCollection
2222
viewer->ReceiveData(packetData);
2323
}
24-
};
24+
}
2525

2626
void DataViewerCollection::RegisterViewer(const std::shared_ptr<IDataViewer>& dataViewer)
2727
{

‎lib/api/DataViewerCollection.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ namespace MAT_NS_BEGIN {
3131

3232
virtual bool IsViewerRegistered(const char* viewerName) const override;
3333

34-
virtual ~DataViewerCollection() noexcept {};
34+
virtual ~DataViewerCollection() noexcept {}
3535
private:
3636
MATSDK_LOG_DECL_COMPONENT_CLASS();
3737

‎lib/api/IRuntimeConfig.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ namespace MAT_NS_BEGIN
192192
/// <returns>Provider Group Id</returns>
193193
virtual const char* GetProviderGroupId() = 0;
194194

195-
virtual ~IRuntimeConfig() {};
195+
virtual ~IRuntimeConfig() {}
196196
};
197197

198198
/// @endcond

‎lib/api/LogManagerImpl.cpp

+6-6
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ namespace MAT_NS_BEGIN
102102
return true;
103103
}
104104

105-
MATSDK_LOG_INST_COMPONENT_CLASS(LogManagerImpl, "EventsSDK.LogManager", "Microsoft Telemetry Client - LogManager class");
105+
MATSDK_LOG_INST_COMPONENT_CLASS(LogManagerImpl, "EventsSDK.LogManager", "Microsoft Telemetry Client - LogManager class")
106106

107107
#if 1
108108
// TODO: integrate Tracing API from v1
@@ -348,7 +348,7 @@ namespace MAT_NS_BEGIN
348348
client->SetMsRootCheck(m_logConfiguration[CFG_MAP_HTTP][CFG_BOOL_HTTP_MS_ROOT_CHECK]);
349349
}
350350
#endif
351-
};
351+
}
352352

353353
LogManagerImpl::~LogManagerImpl() noexcept
354354
{
@@ -507,7 +507,7 @@ namespace MAT_NS_BEGIN
507507
const std::string& LogManagerImpl::GetTransmitProfileName()
508508
{
509509
return TransmitProfiles::getProfile();
510-
};
510+
}
511511

512512
ISemanticContext& LogManagerImpl::GetSemanticContext()
513513
{
@@ -665,7 +665,7 @@ namespace MAT_NS_BEGIN
665665
void LogManagerImpl::AddEventListener(DebugEventType type, DebugEventListener& listener)
666666
{
667667
m_debugEventSource.AddEventListener(type, listener);
668-
};
668+
}
669669

670670
/// <summary>
671671
/// Removes the event listener.
@@ -675,7 +675,7 @@ namespace MAT_NS_BEGIN
675675
void LogManagerImpl::RemoveEventListener(DebugEventType type, DebugEventListener& listener)
676676
{
677677
m_debugEventSource.RemoveEventListener(type, listener);
678-
};
678+
}
679679

680680
/// <summary>
681681
/// Dispatches the event.
@@ -685,7 +685,7 @@ namespace MAT_NS_BEGIN
685685
bool LogManagerImpl::DispatchEvent(DebugEvent evt)
686686
{
687687
return m_debugEventSource.DispatchEvent(std::move(evt));
688-
};
688+
}
689689

690690
/// <summary>Attach cascaded DebugEventSource to forward all events to</summary>
691691
bool LogManagerImpl::AttachEventSource(DebugEventSource& other)

‎lib/api/LogManagerImpl.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ namespace MAT_NS_BEGIN
182182
{
183183
const std::string val(value);
184184
return SetContext(name, val, piiKind);
185-
};
185+
}
186186

187187
virtual inline status_t SetContext(const std::string& name, int8_t value, PiiKind piiKind = PiiKind_None) override
188188
{

‎lib/api/Logger.cpp

+7-7
Original file line numberDiff line numberDiff line change
@@ -170,37 +170,37 @@ namespace MAT_NS_BEGIN
170170
void Logger::SetContext(const std::string& k, const char v[], PiiKind pii)
171171
{
172172
SetContext(k, EventProperty(v, pii));
173-
};
173+
}
174174

175175
void Logger::SetContext(const std::string& k, const std::string& v, PiiKind pii)
176176
{
177177
SetContext(k, EventProperty(v, pii));
178-
};
178+
}
179179

180180
void Logger::SetContext(const std::string& k, double v, PiiKind pii)
181181
{
182182
SetContext(k, EventProperty(v, pii));
183-
};
183+
}
184184

185185
void Logger::SetContext(const std::string& k, int64_t v, PiiKind pii)
186186
{
187187
SetContext(k, EventProperty(v, pii));
188-
};
188+
}
189189

190190
void Logger::SetContext(const std::string& k, time_ticks_t v, PiiKind pii)
191191
{
192192
SetContext(k, EventProperty(v, pii));
193-
};
193+
}
194194

195195
void Logger::SetContext(const std::string& k, GUID_t v, PiiKind pii)
196196
{
197197
SetContext(k, EventProperty(v, pii));
198-
};
198+
}
199199

200200
void Logger::SetContext(const std::string& k, bool v, PiiKind pii)
201201
{
202202
SetContext(k, EventProperty(v, pii));
203-
};
203+
}
204204

205205
// The goal of this method is to rewire the logger instance to any other ISemanticContext issued by SDK.
206206
// 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

+5-5
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ namespace MAT_NS_BEGIN
104104
config(customConfig)
105105
{
106106
Variant::merge_map(*customConfig, *defaultRuntimeConfig());
107-
};
107+
}
108108

109109
virtual ~RuntimeConfig_Default()
110110
{
@@ -125,14 +125,14 @@ namespace MAT_NS_BEGIN
125125
UNREFERENCED_PARAMETER(extension);
126126
UNREFERENCED_PARAMETER(experimentationProject);
127127
UNREFERENCED_PARAMETER(eventName);
128-
};
128+
}
129129

130130
virtual EventLatency GetEventLatency(std::string const& tenantId = std::string(), std::string const& eventName = std::string()) override
131131
{
132132
UNREFERENCED_PARAMETER(tenantId);
133133
UNREFERENCED_PARAMETER(eventName);
134134
return EventLatency_Normal;
135-
};
135+
}
136136

137137
virtual std::string GetMetaStatsTenantToken() override
138138
{
@@ -144,7 +144,7 @@ namespace MAT_NS_BEGIN
144144
return std::string(token);
145145
}
146146
return std::string(defaultToken);
147-
};
147+
}
148148

149149
virtual unsigned GetMetaStatsSendIntervalSec() override
150150
{
@@ -154,7 +154,7 @@ namespace MAT_NS_BEGIN
154154
virtual unsigned GetOfflineStorageMaximumSizeBytes() override
155155
{
156156
return config[CFG_INT_CACHE_FILE_SIZE];
157-
};
157+
}
158158

159159
virtual unsigned GetOfflineStorageResizeThresholdPct() override
160160
{

‎lib/decorators/BaseDecorator.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ namespace MAT_NS_BEGIN
2424

2525
public:
2626
BaseDecorator(ILogManager& owner);
27-
virtual ~BaseDecorator() {};
27+
virtual ~BaseDecorator() {}
2828
bool decorate(CsProtocol::Record& record);
2929

3030
protected:

‎lib/decorators/EventPropertiesDecorator.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ namespace MAT_NS_BEGIN {
5353
//
5454
randomLocalId = "r:";
5555
randomLocalId+= PAL::generateUuidString();
56-
};
56+
}
5757

5858
void dropPiiPartA(::CsProtocol::Record& record)
5959
{

‎lib/decorators/SemanticApiDecorators.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ namespace MAT_NS_BEGIN {
1616
class SemanticApiDecorators : public BaseDecorator {
1717

1818
public:
19-
SemanticApiDecorators(ILogManager& owner) : BaseDecorator(owner) {};
19+
SemanticApiDecorators(ILogManager& owner) : BaseDecorator(owner) {}
2020

2121
bool decorateAggregatedMetricMessage(::CsProtocol::Record& record, AggregatedMetricData const& metricData)
2222
{

‎lib/http/HttpClientFactory.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333

3434
namespace MAT_NS_BEGIN {
3535

36-
MATSDK_LOG_INST_COMPONENT_CLASS(HttpClientFactory, "EventsSDK.HttpClientFactory", "Events telemetry client - HttpClientFactory class");
36+
MATSDK_LOG_INST_COMPONENT_CLASS(HttpClientFactory, "EventsSDK.HttpClientFactory", "Events telemetry client - HttpClientFactory class")
3737

3838
#if defined(MATSDK_PAL_WIN32)
3939
#ifdef _WINRT_DLL

‎lib/http/HttpClientManager.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ namespace MAT_NS_BEGIN {
6161
// as well pass the data back by updating the data structure.
6262
DebugEvent evt(EVT_HTTP_STATE, size_t(state), 0, data, size);
6363
m_hcm.m_logManager.DispatchEvent(evt);
64-
};
64+
}
6565

6666

6767
virtual ~HttpCallback()

‎lib/http/HttpClient_WinInet.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -538,7 +538,7 @@ void HttpClient_WinInet::CancelAllRequests()
538538
PAL::sleep(100);
539539
std::this_thread::yield();
540540
}
541-
};
541+
}
542542

543543
/// <summary>
544544
/// Enforces MS-root server certificate check.

‎lib/include/mat/IDeviceInformation.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ namespace PAL_NS_BEGIN {
2424
{
2525
public:
2626

27-
virtual ~IDeviceInformation() {};
27+
virtual ~IDeviceInformation() {}
2828

2929
/// <summary>
3030
/// Gets the unique ID of the current device

‎lib/include/mat/INetworkInformation.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ namespace PAL_NS_BEGIN {
2727
{
2828
public:
2929

30-
virtual ~INetworkInformation() {};
30+
virtual ~INetworkInformation() {}
3131

3232
/// <summary>
3333
/// Gets the current network provider for the device

‎lib/include/mat/ISystemInformation.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ namespace PAL_NS_BEGIN {
1616
{
1717
public:
1818

19-
virtual ~ISystemInformation() {};
19+
virtual ~ISystemInformation() {}
2020

2121
/// <summary>
2222
/// Gets the App ID.

‎lib/include/public/CAPIClient.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ namespace MAT_NS_BEGIN
4747
{
4848
handle = evt_open(config);
4949
return handle;
50-
};
50+
}
5151

5252
evt_status_t configure(const char* config)
5353
{

‎lib/include/public/CommonFields.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,6 @@
118118
virtual void SETTER_METHOD (name) (const std::string & x) \
119119
{ \
120120
SetCommonField(placeholder, x); \
121-
};
121+
}
122122

123123
#endif

‎lib/include/public/DebugEvents.hpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -158,12 +158,12 @@ namespace MAT_NS_BEGIN
158158
size_t size;
159159

160160
/// <summary>DebugEvent The default DebugEvent constructor.</summary>
161-
DebugEvent() : seq(0), ts(0), type(EVT_UNKNOWN), param1(0), param2(0), data(NULL), size(0) {};
161+
DebugEvent() : seq(0), ts(0), type(EVT_UNKNOWN), param1(0), param2(0), data(NULL), size(0) {}
162162

163-
DebugEvent(DebugEventType type) : seq(0), ts(0), type(type), param1(0), param2(0), data(NULL), size(0) {};
163+
DebugEvent(DebugEventType type) : seq(0), ts(0), type(type), param1(0), param2(0), data(NULL), size(0) {}
164164

165165
DebugEvent(DebugEventType type, size_t param1, size_t param2 = 0, void* data = nullptr, size_t size = 0) :
166-
seq(0), ts(0), type(type), param1(param1), param2(param2), data(data), size(size) {};
166+
seq(0), ts(0), type(type), param1(param1), param2(param2), data(data), size(size) {}
167167
};
168168

169169
/// <summary>

‎lib/include/public/IHttpClient.hpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -494,7 +494,7 @@ namespace MAT_NS_BEGIN
494494
std::ignore = state;
495495
std::ignore = data;
496496
std::ignore = size;
497-
};
497+
}
498498
};
499499

500500
/// <summary>
@@ -542,7 +542,7 @@ namespace MAT_NS_BEGIN
542542
/// <param name="id">A string that contains the ID of the request to cancel.</param>
543543
virtual void CancelRequestAsync(std::string const& id) = 0;
544544

545-
virtual void CancelAllRequests() {};
545+
virtual void CancelAllRequests() {}
546546
};
547547

548548
/// @endcond

‎lib/include/public/IModule.hpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,13 @@ namespace MAT_NS_BEGIN
2727
/// Initializes the module.
2828
/// Invoked as part of parent ILogManager is constructed.
2929
/// </summary>
30-
virtual void Initialize(ILogManager*) noexcept {};
30+
virtual void Initialize(ILogManager*) noexcept {}
3131

3232
/// <summary>
3333
/// Tears down the module.
3434
/// Invoked as part of parent ILogManager's FlushAndTeardown() method.
3535
/// </summary>
36-
virtual void Teardown() noexcept {};
36+
virtual void Teardown() noexcept {}
3737
};
3838

3939
/// @endcond

‎lib/include/public/IOfflineStorage.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,7 @@ namespace MAT_NS_BEGIN {
335335

336336
virtual bool ResizeDb() = 0;
337337

338-
virtual void ReleaseAllRecords() {};
338+
virtual void ReleaseAllRecords() {}
339339

340340
};
341341

0 commit comments

Comments
 (0)