Fixed standard issues

Author: michalChrobot

com.unity.netcode.gameobjects/Runtime/Components/NetworkAnimator.cs

@@ -754,11 +754,8 @@ static AnimationParamEnumWrapper()
         /// </summary>
         private void SpawnCleanup()
         {
-            if (m_NetworkAnimatorStateChangeHandler != null)
-            {
-                m_NetworkAnimatorStateChangeHandler.DeregisterUpdate();
-                m_NetworkAnimatorStateChangeHandler = null;
-            }
+            m_NetworkAnimatorStateChangeHandler?.DeregisterUpdate();
+            m_NetworkAnimatorStateChangeHandler = null;
         }
 
         public override void OnDestroy()

com.unity.netcode.gameobjects/Runtime/Components/RigidbodyContactEventManager.cs

@@ -38,7 +38,7 @@ public interface IContactEventHandler
         /// <summary>
         /// Should return a <see cref="Rigidbody"/>.
         /// </summary>
-        Rigidbody GetRigidbody();
+        public Rigidbody GetRigidbody();
 
         /// <summary>
         /// Invoked by the <see cref="RigidbodyContactEventManager"/> instance.
@@ -49,7 +49,7 @@ public interface IContactEventHandler
         /// <param name="contactPoint">The world space location of the contact event.</param>
         /// <param name="hasCollisionStay">Will be set if this is a collision stay contact event (i.e. it is not the first contact event and continually has contact)</param>
         /// <param name="averagedCollisionStayNormal">The average normal of the collision stay contact over time.</param>
-        void ContactEvent(ulong eventId, Vector3 averagedCollisionNormal, Rigidbody collidingBody, Vector3 contactPoint, bool hasCollisionStay = false, Vector3 averagedCollisionStayNormal = default);
+        public void ContactEvent(ulong eventId, Vector3 averagedCollisionNormal, Rigidbody collidingBody, Vector3 contactPoint, bool hasCollisionStay = false, Vector3 averagedCollisionStayNormal = default);
     }
 
     /// <summary>
@@ -62,7 +62,7 @@ public interface IContactEventHandlerWithInfo : IContactEventHandler
         /// Invoked by <see cref="RigidbodyContactEventManager"/> for each set of contact events it is processing (prior to processing).
         /// </summary>
         /// <returns><see cref="ContactEventHandlerInfo"/></returns>
-        ContactEventHandlerInfo GetContactEventHandlerInfo();
+        public ContactEventHandlerInfo GetContactEventHandlerInfo();
     }
 
     /// <summary>

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

@@ -1623,12 +1623,8 @@ internal void ShutdownInternal()
 
             // Shutdown connection manager last which shuts down transport
             ConnectionManager.Shutdown();
-
-            if (MessageManager != null)
-            {
-                MessageManager.Dispose();
-                MessageManager = null;
-            }
+            MessageManager?.Dispose();
+            MessageManager = null;
 
             // Let the NetworkSceneManager clean up its two SceneEvenData instances
             SceneManager?.Dispose();
@@ -1745,9 +1741,9 @@ private void OnDestroy()
         /// </summary>
         internal interface INetworkManagerHelper
         {
-            bool NotifyUserOfNestedNetworkManager(NetworkManager networkManager, bool ignoreNetworkManagerCache = false, bool editorTest = false);
+            public bool NotifyUserOfNestedNetworkManager(NetworkManager networkManager, bool ignoreNetworkManagerCache = false, bool editorTest = false);
 
-            void CheckAndNotifyUserNetworkObjectRemoved(NetworkManager networkManager, bool editorTest = false);
+            public void CheckAndNotifyUserNetworkObjectRemoved(NetworkManager networkManager, bool editorTest = false);
 
             internal NetcodeAnalytics Analytics();
         }

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

@@ -15,7 +15,7 @@ public interface INetworkUpdateSystem
         /// The update method that is being executed in the context of related <see cref="NetworkUpdateStage"/>.
         /// </summary>
         /// <param name="updateStage">The <see cref="NetworkUpdateStage"/> that is being executed.</param>
-        void NetworkUpdate(NetworkUpdateStage updateStage);
+        public void NetworkUpdate(NetworkUpdateStage updateStage);
     }
 
     /// <summary>

com.unity.netcode.gameobjects/Runtime/Messaging/IDeferredNetworkMessageManager.cs

@@ -18,20 +18,20 @@ internal enum TriggerType
         /// There is a one second maximum lifetime of triggers to avoid memory leaks. After one second has passed
         /// without the requested object ID being spawned, the triggers for it are automatically deleted.
         /// </summary>
-        void DeferMessage(TriggerType trigger, ulong key, FastBufferReader reader, ref NetworkContext context, string messageType = null);
+        public void DeferMessage(TriggerType trigger, ulong key, FastBufferReader reader, ref NetworkContext context, string messageType = null);
 
         /// <summary>
         /// Cleans up any trigger that's existed for more than a second.
         /// These triggers were probably for situations where a request was received after a despawn rather than before a spawn.
         /// </summary>
-        void CleanupStaleTriggers();
+        public void CleanupStaleTriggers();
 
-        void ProcessTriggers(TriggerType trigger, ulong key);
+        public void ProcessTriggers(TriggerType trigger, ulong key);
 
         /// <summary>
         /// Cleans up any trigger that's existed for more than a second.
         /// These triggers were probably for situations where a request was received after a despawn rather than before a spawn.
         /// </summary>
-        void CleanupAllTriggers();
+        public void CleanupAllTriggers();
     }
 }

com.unity.netcode.gameobjects/Runtime/Messaging/INetworkHooks.cs

@@ -15,7 +15,7 @@ internal interface INetworkHooks
         /// <param name="clientId">The destination clientId</param>
         /// <param name="message">The message being sent</param>
         /// <param name="delivery"></param>
-        void OnBeforeSendMessage<T>(ulong clientId, ref T message, NetworkDelivery delivery) where T : INetworkMessage;
+        public void OnBeforeSendMessage<T>(ulong clientId, ref T message, NetworkDelivery delivery) where T : INetworkMessage;
 
         /// <summary>
         /// Called after an individual message is sent.
@@ -24,23 +24,23 @@ internal interface INetworkHooks
         /// <param name="message">The message being sent</param>
         /// <param name="delivery"></param>
         /// <param name="messageSizeBytes">Number of bytes in the message, not including the message header</param>
-        void OnAfterSendMessage<T>(ulong clientId, ref T message, NetworkDelivery delivery, int messageSizeBytes) where T : INetworkMessage;
+        public void OnAfterSendMessage<T>(ulong clientId, ref T message, NetworkDelivery delivery, int messageSizeBytes) where T : INetworkMessage;
 
         /// <summary>
         /// Called before an individual message is received.
         /// </summary>
         /// <param name="senderId">The source clientId</param>
         /// <param name="messageType">The type of the message being sent</param>
         /// <param name="messageSizeBytes">Number of bytes in the message, not including the message header</param>
-        void OnBeforeReceiveMessage(ulong senderId, Type messageType, int messageSizeBytes);
+        public void OnBeforeReceiveMessage(ulong senderId, Type messageType, int messageSizeBytes);
 
         /// <summary>
         /// Called after an individual message is received.
         /// </summary>
         /// <param name="senderId">The source clientId</param>
         /// <param name="messageType">The type of the message being sent</param>
         /// <param name="messageSizeBytes">Number of bytes in the message, not including the message header</param>
-        void OnAfterReceiveMessage(ulong senderId, Type messageType, int messageSizeBytes);
+        public void OnAfterReceiveMessage(ulong senderId, Type messageType, int messageSizeBytes);
 
         /// <summary>
         /// Called before a batch of messages is sent
@@ -49,7 +49,7 @@ internal interface INetworkHooks
         /// <param name="messageCount">Number of messages in the batch</param>
         /// <param name="batchSizeInBytes">Number of bytes in the batch, including the batch header</param>
         /// <param name="delivery"></param>
-        void OnBeforeSendBatch(ulong clientId, int messageCount, int batchSizeInBytes, NetworkDelivery delivery);
+        public void OnBeforeSendBatch(ulong clientId, int messageCount, int batchSizeInBytes, NetworkDelivery delivery);
 
         /// <summary>
         /// Called after a batch of messages is sent
@@ -58,23 +58,23 @@ internal interface INetworkHooks
         /// <param name="messageCount">Number of messages in the batch</param>
         /// <param name="batchSizeInBytes">Number of bytes in the batch, including the batch header</param>
         /// <param name="delivery"></param>
-        void OnAfterSendBatch(ulong clientId, int messageCount, int batchSizeInBytes, NetworkDelivery delivery);
+        public void OnAfterSendBatch(ulong clientId, int messageCount, int batchSizeInBytes, NetworkDelivery delivery);
 
         /// <summary>
         /// Called before a batch of messages is received
         /// </summary>
         /// <param name="senderId">The source clientId</param>
         /// <param name="messageCount">Number of messages in the batch</param>
         /// <param name="batchSizeInBytes">Number of bytes in the batch, including the batch header</param>
-        void OnBeforeReceiveBatch(ulong senderId, int messageCount, int batchSizeInBytes);
+        public void OnBeforeReceiveBatch(ulong senderId, int messageCount, int batchSizeInBytes);
 
         /// <summary>
         /// Called after a batch of messages is received
         /// </summary>
         /// <param name="senderId">The source clientId</param>
         /// <param name="messageCount">Number of messages in the batch</param>
         /// <param name="batchSizeInBytes">Number of bytes in the batch, including the batch header</param>
-        void OnAfterReceiveBatch(ulong senderId, int messageCount, int batchSizeInBytes);
+        public void OnAfterReceiveBatch(ulong senderId, int messageCount, int batchSizeInBytes);
 
 
         /// <summary>
@@ -84,7 +84,7 @@ internal interface INetworkHooks
         /// <param name="messageType">The type of the message</param>
         /// <param name="delivery"></param>
         /// <returns></returns>
-        bool OnVerifyCanSend(ulong destinationId, Type messageType, NetworkDelivery delivery);
+        public bool OnVerifyCanSend(ulong destinationId, Type messageType, NetworkDelivery delivery);
 
         /// <summary>
         /// Called before a message is received. If this returns false, the message will be discarded.
@@ -94,7 +94,7 @@ internal interface INetworkHooks
         /// <param name="messageContent">The FastBufferReader containing the message</param>
         /// <param name="context">The NetworkContext the message is being processed in</param>
         /// <returns></returns>
-        bool OnVerifyCanReceive(ulong senderId, Type messageType, FastBufferReader messageContent, ref NetworkContext context);
+        public bool OnVerifyCanReceive(ulong senderId, Type messageType, FastBufferReader messageContent, ref NetworkContext context);
 
         /// <summary>
         /// Called after a message is serialized, but before it's handled.
@@ -103,7 +103,7 @@ internal interface INetworkHooks
         /// <param name="message">The message object</param>
         /// <param name="context">The network context the message is being ahandled in</param>
         /// <typeparam name="T"></typeparam>
-        void OnBeforeHandleMessage<T>(ref T message, ref NetworkContext context) where T : INetworkMessage;
+        public void OnBeforeHandleMessage<T>(ref T message, ref NetworkContext context) where T : INetworkMessage;
 
         /// <summary>
         /// Called after a message is serialized and handled.
@@ -112,6 +112,6 @@ internal interface INetworkHooks
         /// <param name="message">The message object</param>
         /// <param name="context">The network context the message is being ahandled in</param>
         /// <typeparam name="T"></typeparam>
-        void OnAfterHandleMessage<T>(ref T message, ref NetworkContext context) where T : INetworkMessage;
+        public void OnAfterHandleMessage<T>(ref T message, ref NetworkContext context) where T : INetworkMessage;
     }
 }

com.unity.netcode.gameobjects/Runtime/Messaging/INetworkMessage.cs

@@ -39,10 +39,10 @@ namespace Unity.Netcode
     /// </summary>
     internal interface INetworkMessage
     {
-        void Serialize(FastBufferWriter writer, int targetVersion);
-        bool Deserialize(FastBufferReader reader, ref NetworkContext context, int receivedMessageVersion);
-        void Handle(ref NetworkContext context);
-        int Version { get; }
+        public void Serialize(FastBufferWriter writer, int targetVersion);
+        public bool Deserialize(FastBufferReader reader, ref NetworkContext context, int receivedMessageVersion);
+        public void Handle(ref NetworkContext context);
+        public int Version { get; }
     }
 
 

com.unity.netcode.gameobjects/Runtime/Messaging/INetworkMessageProvider.cs

@@ -4,6 +4,6 @@ namespace Unity.Netcode
 {
     internal interface INetworkMessageProvider
     {
-        List<NetworkMessageManager.MessageWithHandler> GetMessages();
+        public List<NetworkMessageManager.MessageWithHandler> GetMessages();
     }
 }

com.unity.netcode.gameobjects/Runtime/Messaging/INetworkMessageSender.cs

@@ -2,6 +2,6 @@ namespace Unity.Netcode
 {
     internal interface INetworkMessageSender
     {
-        void Send(ulong clientId, NetworkDelivery delivery, FastBufferWriter batchData);
+        public void Send(ulong clientId, NetworkDelivery delivery, FastBufferWriter batchData);
     }
 }

com.unity.netcode.gameobjects/Runtime/Messaging/RpcTargets/IGroupRpcTarget.cs

@@ -2,8 +2,8 @@ namespace Unity.Netcode
 {
     internal interface IGroupRpcTarget
     {
-        void Add(ulong clientId);
-        void Clear();
-        BaseRpcTarget Target { get; }
+        public void Add(ulong clientId);
+        public void Clear();
+        public BaseRpcTarget Target { get; }
     }
 }