diff options
| author | Cody Robibero <cody@robibe.ro> | 2021-12-24 02:01:06 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-12-24 02:01:06 +0000 |
| commit | cecfdeeec3effa61de2a83b5656059c08536d94e (patch) | |
| tree | b77ae70228a2f62d12b258e9467721bdc4bf44fc /src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs | |
| parent | fcf5b9b46e7d120904fdafb3df726b1a1309660d (diff) | |
| parent | 076a13abeb344f7a79d81cce030799c8e77dbfaf (diff) | |
Merge branch 'master' into unharden-for-lxc
Diffstat (limited to 'src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs')
| -rw-r--r-- | src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs | 20 |
1 files changed, 4 insertions, 16 deletions
diff --git a/src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs b/src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs index 6de238b39..28437023f 100644 --- a/src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs +++ b/src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs @@ -15,13 +15,10 @@ namespace Jellyfin.Extensions.Json.Converters /// <inheritdoc /> public override TStruct? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options) { - if (reader.TokenType == JsonTokenType.Null) - { - return null; - } - // Token is empty string. - if (reader.TokenType == JsonTokenType.String && ((reader.HasValueSequence && reader.ValueSequence.IsEmpty) || reader.ValueSpan.IsEmpty)) + if (reader.TokenType == JsonTokenType.String + && ((reader.HasValueSequence && reader.ValueSequence.IsEmpty) + || (!reader.HasValueSequence && reader.ValueSpan.IsEmpty))) { return null; } @@ -31,15 +28,6 @@ namespace Jellyfin.Extensions.Json.Converters /// <inheritdoc /> public override void Write(Utf8JsonWriter writer, TStruct? value, JsonSerializerOptions options) - { - if (value.HasValue) - { - JsonSerializer.Serialize(writer, value.Value, options); - } - else - { - writer.WriteNullValue(); - } - } + => JsonSerializer.Serialize(writer, value!.Value, options); // null got handled higher up the call stack } } |
