aboutsummaryrefslogtreecommitdiff
path: root/src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs
diff options
context:
space:
mode:
authorCody Robibero <cody@robibe.ro>2021-12-24 02:01:06 +0000
committerGitHub <noreply@github.com>2021-12-24 02:01:06 +0000
commitcecfdeeec3effa61de2a83b5656059c08536d94e (patch)
treeb77ae70228a2f62d12b258e9467721bdc4bf44fc /src/Jellyfin.Extensions/Json/Converters/JsonNullableStructConverter.cs
parentfcf5b9b46e7d120904fdafb3df726b1a1309660d (diff)
parent076a13abeb344f7a79d81cce030799c8e77dbfaf (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.cs20
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
}
}