feat: Optimize destroy object message

Author: EmandM

com.unity.netcode.gameobjects/Runtime/Messaging/Messages/DestroyObjectMessage.cs

@@ -5,7 +5,8 @@ namespace Unity.Netcode
 {
     internal struct DestroyObjectMessage : INetworkMessage, INetworkSerializeByMemcpy
     {
-        public int Version => 0;
+        private const int k_OptimizeDestroyObjectMessage = 1;
+        public int Version => k_OptimizeDestroyObjectMessage;
 
         private const string k_Name = "DestroyObjectMessage";
 
@@ -19,19 +20,21 @@ internal struct DestroyObjectMessage : INetworkMessage, INetworkSerializeByMemcp
 
         internal bool IsDistributedAuthority;
 
-        internal const byte ClientTargetedDestroy = 0x01;
+        private const byte k_ClientTargetedDestroy = 0x01;
+        private const byte k_DeferredDespawn = 0x02;
 
         internal bool IsTargetedDestroy
         {
-            get
-            {
-                return GetFlag(ClientTargetedDestroy);
-            }
+            get => GetFlag(k_ClientTargetedDestroy);
 
-            set
-            {
-                SetFlag(value, ClientTargetedDestroy);
-            }
+            set => SetFlag(value, k_ClientTargetedDestroy);
+        }
+
+        internal bool IsDeferredDespawn
+        {
+            get => GetFlag(k_DeferredDespawn);
+
+            set => SetFlag(value, k_DeferredDespawn);
         }
 
         private bool GetFlag(int flag)
@@ -47,7 +50,11 @@ private void SetFlag(bool set, byte flag)
 
         public void Serialize(FastBufferWriter writer, int targetVersion)
         {
+            // Set deferred despawn flag
+            IsDeferredDespawn = DeferredDespawnTick > 0;
+
             BytePacker.WriteValueBitPacked(writer, NetworkObjectId);
+
             if (IsDistributedAuthority)
             {
                 writer.WriteByteSafe(m_DestroyFlags);
@@ -56,9 +63,17 @@ public void Serialize(FastBufferWriter writer, int targetVersion)
                 {
                     BytePacker.WriteValueBitPacked(writer, TargetClientId);
                 }
-                BytePacker.WriteValueBitPacked(writer, DeferredDespawnTick);
+
+                if (targetVersion < k_OptimizeDestroyObjectMessage || IsDeferredDespawn)
+                {
+                    BytePacker.WriteValueBitPacked(writer, DeferredDespawnTick);
+                }
+            }
+
+            if (targetVersion < k_OptimizeDestroyObjectMessage)
+            {
+                writer.WriteValueSafe(DestroyGameObject);
             }
-            writer.WriteValueSafe(DestroyGameObject);
         }
 
         public bool Deserialize(FastBufferReader reader, ref NetworkContext context, int receivedMessageVersion)
@@ -77,18 +92,27 @@ public bool Deserialize(FastBufferReader reader, ref NetworkContext context, int
                 {
                     ByteUnpacker.ReadValueBitPacked(reader, out TargetClientId);
                 }
-                ByteUnpacker.ReadValueBitPacked(reader, out DeferredDespawnTick);
+
+                if (receivedMessageVersion < k_OptimizeDestroyObjectMessage || IsDeferredDespawn)
+                {
+                    ByteUnpacker.ReadValueBitPacked(reader, out DeferredDespawnTick);
+                }
             }
 
-            reader.ReadValueSafe(out DestroyGameObject);
+            if (receivedMessageVersion < k_OptimizeDestroyObjectMessage)
+            {
+                reader.ReadValueSafe(out DestroyGameObject);
+            }
 
-            if (!networkManager.SpawnManager.SpawnedObjects.ContainsKey(NetworkObjectId))
+            if (networkManager.SpawnManager.SpawnedObjects.ContainsKey(NetworkObjectId))
             {
-                // Client-Server mode we always defer where in distributed authority mode we only defer if it is not a targeted destroy
-                if (!networkManager.DistributedAuthorityMode || (networkManager.DistributedAuthorityMode && !IsTargetedDestroy))
-                {
-                    networkManager.DeferredMessageManager.DeferMessage(IDeferredNetworkMessageManager.TriggerType.OnSpawn, NetworkObjectId, reader, ref context, k_Name);
-                }
+                return true;
+            }
+
+            // Client-Server mode we always defer where in distributed authority mode we only defer if it is not a targeted destroy
+            if (!networkManager.DistributedAuthorityMode || (networkManager.DistributedAuthorityMode && !IsTargetedDestroy))
+            {
+                networkManager.DeferredMessageManager.DeferMessage(IDeferredNetworkMessageManager.TriggerType.OnSpawn, NetworkObjectId, reader, ref context, k_Name);
             }
             return true;
         }