style

Updating some comments based on AI suggestions.

Author: NoelStephensUnity

com.unity.netcode.gameobjects/Runtime/Components/Helpers/AttachableBehaviour.cs

@@ -467,6 +467,8 @@ public void Attach(AttachableNode attachableNode)
         /// </summary>
         internal void InternalDetach()
         {
+            // If this instance is not in the middle of being destroyed, the attachable node is not null, and the node is not destroying
+            // =or= the scene it is located in is in the middle of being unloaded, then re-parent under the default parent.
             if (!IsDestroying && m_AttachableNode && (!m_AttachableNode.IsDestroying || m_AttachableNode.gameObject.scene.isLoaded))
             {
                 if (m_DefaultParent)

com.unity.netcode.gameobjects/Runtime/Core/NetworkBehaviour.cs

@@ -644,11 +644,10 @@ protected NetworkBehaviour GetNetworkBehaviour(ushort behaviourId)
         public ulong OwnerClientId { get; internal set; }
 
         /// <summary>
-        /// Returns true if the NetworkObject is in the middle of being destroyed or
-        /// if there is no valid assigned NetworkObject.
+        /// Returns true if the NetworkObject is in the middle of being destroyed.
         /// </summary>
         /// <remarks>
-        /// <see cref="NetworkObject.IsDestroying"/>
+        /// <see cref="SetIsDestroying"/>
         /// </remarks>
         internal bool IsDestroying { get; private set; }
 
@@ -675,6 +674,7 @@ protected internal virtual void OnIsDestroying()
         /// </remarks>
         internal void SetIsDestroying()
         {
+            // We intentionally invoke this before setting the IsDestroying flag.
             OnIsDestroying();
             IsDestroying = true;
         }