fix: make sure resolved typedef isn't null (#731)

Lymdun · web-flow · commit 2f6414f7f676 · 2021-03-26T13:40:25.000-05:00
* fix: make sure resolved typedef isn't null

* re-use variable
diff --git a/Assets/Mirage/Weaver/Processors/SyncObjectProcessor.cs b/Assets/Mirage/Weaver/Processors/SyncObjectProcessor.cs
@@ -36,7 +36,13 @@ public void ProcessSyncObjects(TypeDefinition td)
                     continue;
                 }
 
-                if (fd.FieldType.Resolve().ImplementsInterface<ISyncObject>())
+                TypeDefinition tf = fd.FieldType.Resolve();
+                if (tf == null)
+                {
+                    continue;
+                }
+
+                if (tf.ImplementsInterface<ISyncObject>())
                 {
                     if (fd.IsStatic)
                     {

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,13 @@ public void ProcessSyncObjects(TypeDefinition td)`
`36`	`36`	`continue;`
`37`	`37`	`}`
`38`	`38`
`39`		`- if (fd.FieldType.Resolve().ImplementsInterface<ISyncObject>())`
	`39`	`+ TypeDefinition tf = fd.FieldType.Resolve();`
	`40`	`+ if (tf == null)`
	`41`	`+ {`
	`42`	`+ continue;`
	`43`	`+ }`
	`44`	`+`
	`45`	`+ if (tf.ImplementsInterface<ISyncObject>())`
`40`	`46`	`{`
`41`	`47`	`if (fd.IsStatic)`
`42`	`48`	`{`