mirrors/dotnet-installer
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Simplified combining dictionaries in Telemetry

Browse source
This commit is contained in:
Petr Onderka 2018-05-31 01:03:27 +02:00 committed by GitHub
parent dae5099ffb
commit 139fc86dbc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/dotnet/Telemetry/Telemetry.cs
View file

@ -140,16 +140,9 @@ namespace Microsoft.DotNet.Cli.Telemetry
if (measurements != null) if (measurements != null)
{ {
foreach (KeyValuePair<string, double> measurement in measurements) foreach (KeyValuePair<string, double> measurement in measurements)
{
if (eventMeasurements.ContainsKey(measurement.Key))
{ {
eventMeasurements[measurement.Key] = measurement.Value; eventMeasurements[measurement.Key] = measurement.Value;
} }
else
{
eventMeasurements.Add(measurement.Key, measurement.Value);
}
}
} }
return eventMeasurements; return eventMeasurements;
} }
@ -160,16 +153,9 @@ namespace Microsoft.DotNet.Cli.Telemetry
{ {
var eventProperties = new Dictionary<string, string>(_commonProperties); var eventProperties = new Dictionary<string, string>(_commonProperties);
foreach (KeyValuePair<string, string> property in properties) foreach (KeyValuePair<string, string> property in properties)
{
if (eventProperties.ContainsKey(property.Key))
{ {
eventProperties[property.Key] = property.Value; eventProperties[property.Key] = property.Value;
} }
else
{
eventProperties.Add(property.Key, property.Value);
}
}
return eventProperties; return eventProperties;
} }
else else