[unitytls] UnityTls.IsSupported no longer relies on exception handling but on a simple null check

Author: AndreasReich

mcs/class/System/Mono.Net.Security/MonoTlsProviderFactory.cs

@@ -228,7 +228,7 @@ static void InitializeProviderRegistration ()
 				providerCache = new Dictionary<Guid,MSI.MonoTlsProvider> ();
 
 #if UNITY
-				if (Mono.Unity.UnityTls.IsSupported())
+				if (Mono.Unity.UnityTls.IsSupported)
 				{
 					var unityTlsEntry = new Tuple<Guid,String> (UnityTlsId, "Mono.Unity.UnityTlsProvider");
 					providerRegistration.Add ("default", unityTlsEntry);

mcs/class/System/Mono.UnityTls/UnityTls.cs

@@ -235,21 +235,18 @@ public class mono_unity_unitytls_interface
 
         private static mono_unity_unitytls_interface marshalledInterface = null;
 
-        public static bool IsSupported()
-        {
-            try {
-                return NativeInterface != null;
-            } catch (System.Exception) {
-                return false;
-            }
-        }
+        public static bool IsSupported => NativeInterface != null;
 
         public static mono_unity_unitytls_interface NativeInterface
         {
             get
             {
-                if (marshalledInterface == null)
-                    marshalledInterface = Marshal.PtrToStructure<mono_unity_unitytls_interface>(mono_unity_get_unitytls_interface());
+                if (marshalledInterface == null) {
+                    IntPtr rawInterface = mono_unity_get_unitytls_interface ();
+                    if (rawInterface == IntPtr.Zero)
+                        return null;
+                    marshalledInterface = Marshal.PtrToStructure<mono_unity_unitytls_interface> (rawInterface);
+                }
                 return marshalledInterface;
             }
         }