chore: remove #region blocks (non-standard)

Author: 0xFA11

com.unity.netcode.gameobjects/Runtime/Configuration/NetworkConfig.cs

@@ -281,8 +281,6 @@ internal void InitializePrefabs()
             Prefabs.Initialize();
         }
 
-        #region Legacy Network Prefab List
-
         [NonSerialized]
         private bool m_DidWarnOldPrefabList = false;
 
@@ -342,7 +340,5 @@ internal NetworkPrefabsList MigrateOldNetworkPrefabsToNetworkPrefabsList()
         [FormerlySerializedAs("NetworkPrefabs")]
         [SerializeField]
         internal List<NetworkPrefab> OldPrefabList;
-
-        #endregion
     }
 }

com.unity.netcode.gameobjects/Tests/Editor/Serialization/BaseFastBufferReaderWriterTest.cs

@@ -7,8 +7,6 @@ namespace Unity.Netcode.EditorTests
 {
     public abstract class BaseFastBufferReaderWriterTest
     {
-
-        #region Test Types
         protected enum ByteEnum : byte
         {
             A,
@@ -78,7 +76,6 @@ public enum WriteType
             WriteDirect,
             WriteSafe
         }
-        #endregion
 
         protected abstract void RunTypeTest<T>(T valueToTest) where T : unmanaged;
 
@@ -88,7 +85,6 @@ public enum WriteType
 
         protected abstract void RunTypeArrayTestSafe<T>(T[] valueToTest) where T : unmanaged;
 
-        #region Helpers
         protected TestStruct GetTestStruct()
         {
             var random = new Random();
@@ -111,9 +107,6 @@ protected TestStruct GetTestStruct()
             return testStruct;
         }
 
-        #endregion
-
-
         private void RunTestWithWriteType<T>(T val, WriteType wt, FastBufferWriter.ForPrimitives _ = default) where T : unmanaged
         {
             switch (wt)

com.unity.netcode.gameobjects/Tests/Editor/Serialization/BytePackerTests.cs

@@ -10,8 +10,6 @@ namespace Unity.Netcode.EditorTests
 {
     public class BytePackerTests
     {
-        #region Test Types
-
         private enum ByteEnum : byte
         {
             A,
@@ -74,8 +72,6 @@ public enum WriteType
             WriteAsObject
         }
 
-        #endregion
-
         private unsafe void VerifyBytewiseEquality<T>(T value, T otherValue) where T : unmanaged
         {
             byte* asBytePointer = (byte*)&value;

com.unity.netcode.gameobjects/Tests/Editor/Serialization/FastBufferReaderTests.cs

@@ -9,7 +9,6 @@ namespace Unity.Netcode.EditorTests
 {
     public class FastBufferReaderTests : BaseFastBufferReaderWriterTest
     {
-        #region Common Checks
         private void WriteCheckBytes(FastBufferWriter writer, int writeSize, string failMessage = "")
         {
             Assert.IsTrue(writer.TryBeginWrite(2), "Writer denied write permission");
@@ -230,9 +229,7 @@ private void RunReadMethod<T>(string methodName, FastBufferReader reader, out T[
             method.Invoke(reader, args);
             value = (T[])args[0];
         }
-        #endregion
 
-        #region Generic Checks
         protected override unsafe void RunTypeTest<T>(T valueToTest)
         {
             var writeSize = FastBufferWriter.GetWriteSize(valueToTest);
@@ -343,9 +340,6 @@ protected override unsafe void RunTypeArrayTestSafe<T>(T[] valueToTest)
             }
         }
 
-        #endregion
-
-        #region Tests
         [Test]
         public void GivenFastBufferWriterContainingValue_WhenReadingUnmanagedType_ValueMatchesWhatWasWritten(
             [Values(typeof(byte), typeof(sbyte), typeof(short), typeof(ushort), typeof(int), typeof(uint),
@@ -1220,7 +1214,5 @@ public unsafe void WhenCallingTryBeginReadInternal_AllowedReadPositionDoesNotMov
                 Assert.AreEqual(reader.Handle->AllowedReadMark, 25);
             }
         }
-
-        #endregion
     }
 }

com.unity.netcode.gameobjects/Tests/Editor/Serialization/FastBufferWriterTests.cs

@@ -9,9 +9,6 @@ namespace Unity.Netcode.EditorTests
 {
     public class FastBufferWriterTests : BaseFastBufferReaderWriterTest
     {
-
-        #region Common Checks
-
         private void WriteCheckBytes(FastBufferWriter writer, int writeSize, string failMessage = "")
         {
             Assert.IsTrue(writer.TryBeginWrite(2), "Writer denied write permission");
@@ -66,10 +63,6 @@ private unsafe void CommonChecks<T>(FastBufferWriter writer, T valueToTest, int
             VerifyTypedEquality(valueToTest, writer.GetUnsafePtr());
         }
 
-        #endregion
-
-        #region Generic Checks
-
         private void RunMethod<T>(string methodName, FastBufferWriter writer, in T value) where T : unmanaged
         {
             MethodInfo method = typeof(FastBufferWriter).GetMethod(methodName, new[] { typeof(T).MakeByRefType() });
@@ -248,11 +241,8 @@ protected override unsafe void RunTypeArrayTestSafe<T>(T[] valueToTest)
                 VerifyCheckBytes(underlyingArray, writeSize);
             }
         }
-        #endregion
-
 
-        #region Tests
-        [Test, Description("Tests ")]
+        [Test, Description("Tests")]
         public void WhenWritingUnmanagedType_ValueIsWrittenCorrectly(
             [Values(typeof(byte), typeof(sbyte), typeof(short), typeof(ushort), typeof(int), typeof(uint),
                 typeof(long), typeof(ulong), typeof(bool), typeof(char), typeof(float), typeof(double),
@@ -1317,6 +1307,5 @@ public unsafe void WhenCallingTryBeginWriteInternal_AllowedWritePositionDoesNotM
                 Assert.AreEqual(writer.Handle->AllowedWriteMark, 25);
             }
         }
-        #endregion
     }
 }

com.unity.netcode.gameobjects/Tests/Runtime/NetworkVariableTests.cs

@@ -1010,7 +1010,6 @@ public IEnumerator TestINetworkSerializableStructCallsNetworkSerialize([Values(t
             yield return WaitForConditionOrTimeOut(VerifyCallback);
         }
 
-        #region COULD_BE_REMOVED
         [UnityTest]
         [Ignore("This is used several times already in the NetworkListPredicate")]
         // TODO: If we end up using the new suggested pattern, then delete this
@@ -1221,7 +1220,6 @@ public void TestUnmnagedINetworkSerializableNetworkVariablesDeserializeInPlace()
             Assert.AreEqual(2, variable.Value.Int, "Int was not correctly deserialized");
             Assert.AreEqual("three", variable.Value.Str, "Str was not correctly deserialized");
         }
-        #endregion
 
         private float m_OriginalTimeScale = 1.0f;