Remove comment on OverflowException

angularsen · web-flow · commit d4697073dc22 · 2018-02-20T21:33:12.000+01:00
As per @gojanpaolo 's comment: #389 (comment) This will not throw when converting to double, since double has a big enough range to contain all other numeric types in .NET.
diff --git a/UnitsNet.Serialization.JsonNet/UnitsNetJsonConverter.cs b/UnitsNet.Serialization.JsonNet/UnitsNetJsonConverter.cs
@@ -206,7 +206,6 @@ public override void WriteJson(JsonWriter writer, object obj, JsonSerializer ser
 
             serializer.Serialize(writer, new ValueUnit
             {
-                // This might throw OverflowException for very large values?
                 // TODO Should we serialize long, decimal and long differently?
                 Value = Convert.ToDouble(quantityValue),
                 Unit = quantityUnitName

Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,6 @@ public override void WriteJson(JsonWriter writer, object obj, JsonSerializer ser`
`206`	`206`
`207`	`207`	`serializer.Serialize(writer, new ValueUnit`
`208`	`208`	`{`
`209`		`- // This might throw OverflowException for very large values?`
`210`	`209`	`// TODO Should we serialize long, decimal and long differently?`
`211`	`210`	`Value = Convert.ToDouble(quantityValue),`
`212`	`211`	`Unit = quantityUnitName`