Gittes-archive/AssetRipper_AssetRipper
2
0
Fork 0
mirror of https://github.com/AssetRipper/AssetRipper.git synced 2025-12-11 20:15:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

Handle null in SerializationLogic

Browse Source
This commit is contained in:
Jeremy Pritts 2023-07-21 02:00:26 -04:00
parent f9066cf291
commit 23fabd9b86
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Source/AssetRipper.SerializationLogic/FieldSerializationLogic.cs
View File

@ -317,12 +317,8 @@ namespace AssetRipper.SerializationLogic
public static int GetFixedBufferLength(FieldDefinition fieldDefinition)
{
CustomAttribute? fixedBufferAttribute = GetFixedBufferAttribute(fieldDefinition);
if (fixedBufferAttribute == null)
{
throw new ArgumentException(string.Format("Field '{0}' is not a fixed buffer field.", fieldDefinition.FullName));
}
CustomAttribute fixedBufferAttribute = GetFixedBufferAttribute(fieldDefinition)
?? throw new ArgumentException($"Field '{fieldDefinition.FullName}' is not a fixed buffer field.");
int size = (int)(fixedBufferAttribute.Signature?.FixedArguments[1].Element ?? 0);
@ -391,7 +387,7 @@ namespace AssetRipper.SerializationLogic
return EngineTypePredicates.IsUnityEngineObject(typeReference);
}
public static bool IsNonSerialized(ITypeDescriptor typeDeclaration)
public static bool IsNonSerialized([NotNullWhen(false)] ITypeDescriptor? typeDeclaration)
{
if (typeDeclaration == null)
{
@ -429,7 +425,7 @@ namespace AssetRipper.SerializationLogic
return typeDeclaration.FullName.StartsWith("System."); //can this be done better?
}
public static bool ShouldImplementIDeserializable(ITypeDescriptor typeDeclaration)
public static bool ShouldImplementIDeserializable([NotNullWhen(true)] ITypeDescriptor? typeDeclaration)
{
if (typeDeclaration is { Namespace: EngineTypePredicates.UnityEngineNamespace, Name: "ExposedReference`1" })
{