Cache IsSupported result to avoid redundant checks

xrdevrob · xrdevrob · commit 2d52885b716e · 2025-07-18T10:44:13.000+02:00
diff --git a/Unity-QuestVisionKit/Assets/PassthroughCameraApiSamples/PassthroughCamera/Scripts/PassthroughCameraUtils.cs b/Unity-QuestVisionKit/Assets/PassthroughCameraApiSamples/PassthroughCamera/Scripts/PassthroughCameraUtils.cs
@@ -63,9 +63,9 @@ public static bool IsSupported
                 if (!s_isSupported.HasValue)
                 {
                     var headset = OVRPlugin.GetSystemHeadsetType();
-                    return (headset == OVRPlugin.SystemHeadset.Meta_Quest_3 ||
-                            headset == OVRPlugin.SystemHeadset.Meta_Quest_3S) &&
-                           (!HorizonOSVersion.HasValue || HorizonOSVersion >= MINSUPPORTOSVERSION);
+                    s_isSupported = (headset == OVRPlugin.SystemHeadset.Meta_Quest_3 ||
+                                     headset == OVRPlugin.SystemHeadset.Meta_Quest_3S) &&
+                                    (!HorizonOSVersion.HasValue || HorizonOSVersion >= MINSUPPORTOSVERSION);
                 }
 
                 return s_isSupported.Value;

Original file line number	Diff line number	Diff line change
`@@ -63,9 +63,9 @@ public static bool IsSupported`
`63`	`63`	`if (!s_isSupported.HasValue)`
`64`	`64`	`{`
`65`	`65`	`var headset = OVRPlugin.GetSystemHeadsetType();`
`66`		`- return (headset == OVRPlugin.SystemHeadset.Meta_Quest_3 \|\|`
`67`		`- headset == OVRPlugin.SystemHeadset.Meta_Quest_3S) &&`
`68`		`- (!HorizonOSVersion.HasValue \|\| HorizonOSVersion >= MINSUPPORTOSVERSION);`
	`66`	`+ s_isSupported = (headset == OVRPlugin.SystemHeadset.Meta_Quest_3 \|\|`
	`67`	`+ headset == OVRPlugin.SystemHeadset.Meta_Quest_3S) &&`
	`68`	`+ (!HorizonOSVersion.HasValue \|\| HorizonOSVersion >= MINSUPPORTOSVERSION);`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`return s_isSupported.Value;`