Merge remote-tracking branch 'origin/main' into no-normals-problems

Author: kring

CHANGES.md

@@ -4,9 +4,11 @@
 
 ##### Fixes :wrench:
 
+- Fixed a bug that could cause a crash on AppDomain reloads.
 - Fixed a bug that could cause a crash or incorrect textures when multiple `Cesium3DTileset` tiles referenced the same image by URL.
 - Fixed a bug that could cause incorrect colors in a model that did have vertex colors but did not have normals.
 - Fixed a bug that could cause a hang when attempting to load a model with UINT16 indices where generating flat normals required more than 2^16 vertices.
+- Fixed a bug in the Abseil vcpkg overlay port that could cause linker errors on some systems.
 
 ## v1.14.0 - 2024-12-02
 

Reinterop~/CSharpObjectHandleUtility.cs

@@ -37,8 +37,15 @@ public static IntPtr CopyHandle(IntPtr handle)
                             return handle;
 
                         // Allocate a new GCHandle pointing to the same object.
-                        GCHandle gcHandle = GCHandle.FromIntPtr(handle);
-                        return GCHandle.ToIntPtr(GCHandle.Alloc(gcHandle.Target));
+                        try
+                        {
+                            GCHandle gcHandle = GCHandle.FromIntPtr(handle);
+                            return GCHandle.ToIntPtr(GCHandle.Alloc(gcHandle.Target));
+                        }
+                        catch (Exception)
+                        {
+                            return IntPtr.Zero;
+                        }
                     }
 
                     public static void FreeHandle(IntPtr handle)
@@ -50,14 +57,13 @@ public static void FreeHandle(IntPtr handle)
                         {
                             GCHandle.FromIntPtr(handle).Free();
                         }
-                        catch (ArgumentException e)
+                        catch (Exception)
                         {
                             // The "GCHandle value belongs to a different domain" exception tends
                             // to happen on AppDomain reload, which is common in Unity.
                             // Catch the exception to prevent it propagating through our native
                             // code and blowing things up.
                             // See: https://github.com/CesiumGS/cesium-unity/issues/18
-                            System.Console.WriteLine(e.ToString());
                         }
                     }
 
@@ -66,18 +72,32 @@ public static object GetObjectFromHandle(IntPtr handle)
                         if (handle == IntPtr.Zero)
                             return null;
 
-                        return GCHandle.FromIntPtr(handle).Target;
+                        try
+                        {
+                            return GCHandle.FromIntPtr(handle).Target;
+                        }
+                        catch (Exception)
+                        {
+                            return null;
+                        }
                     }
 
                     public static object GetObjectAndFreeHandle(IntPtr handle)
                     {
                         if (handle == IntPtr.Zero)
                             return null;
 
-                        GCHandle gcHandle = GCHandle.FromIntPtr(handle);
-                        object result = gcHandle.Target;
-                        gcHandle.Free();
-                        return result;
+                        try
+                        {
+                            GCHandle gcHandle = GCHandle.FromIntPtr(handle);
+                            object result = gcHandle.Target;
+                            gcHandle.Free();
+                            return result;
+                        }
+                        catch (Exception)
+                        {
+                            return null;
+                        }
                     }
 
                     public static void ResetHandleObject(IntPtr handle, object newValue)