Make TelemetryValue always required when using DefaultResult

Author: andrewlock

tracer/src/Datadog.Trace/Configuration/ConfigurationSources/Telemetry/DefaultResult.cs

@@ -4,32 +4,18 @@
 // </copyright>
 
 #nullable enable
-using System.Collections.Generic;
 
 namespace Datadog.Trace.Configuration.ConfigurationSources.Telemetry;
 
-internal readonly record struct DefaultResult<T>
+internal readonly struct DefaultResult<T>(T result, string telemetryValue)
 {
-    private readonly string? _telemetryValue;
-
-    public DefaultResult(T result, string? telemetryValue)
-    {
-        Result = result;
-        _telemetryValue = telemetryValue;
-    }
-
     /// <summary>
     /// Gets the value to use as the default result
     /// </summary>
-    public T Result { get; }
+    public T Result { get; } = result;
 
     /// <summary>
     /// Gets a string representation of the result to use in telemetry.
     /// </summary>
-    public string? TelemetryValue => _telemetryValue ?? Result?.ToString();
-
-    public static implicit operator DefaultResult<T>(T result)
-        => result is IDictionary<string, string>
-               ? new(result, telemetryValue: string.Empty) // we don't want to call ToString() on these
-               : new(result, null);
+    public string TelemetryValue { get; } = telemetryValue;
 }