Optimize empty arrays usages (#1490)

Author: hazzik

src/NHibernate.DomainModel/Async/CustomPersister.cs

@@ -21,6 +21,7 @@
 using NHibernate.Tuple.Entity;
 using NHibernate.Type;
 using NHibernate.Util;
+using Array = System.Array;
 
 namespace NHibernate.DomainModel
 {

src/NHibernate.DomainModel/Baz.cs

@@ -396,7 +396,7 @@ public void SetDefaults()
 			StringArray = StringSet.ToArray();
 			StringList = new List<string>(StringArray);
 			IntArray = new int[] {1, 3, 3, 7};
-			FooArray = new Foo[0];
+			FooArray = Array.Empty<Foo>();
 
 			Customs = new List<string[]>();
 			Customs.Add(new String[] {"foo", "bar"});

src/NHibernate.DomainModel/CustomPersister.cs

@@ -11,6 +11,7 @@
 using NHibernate.Tuple.Entity;
 using NHibernate.Type;
 using NHibernate.Util;
+using Array = System.Array;
 
 namespace NHibernate.DomainModel
 {
@@ -130,12 +131,12 @@ public bool[] PropertyInsertability
 
 		public ValueInclusion[] PropertyInsertGenerationInclusions
 		{
-			get { return new ValueInclusion[0]; }
+			get { return Array.Empty<ValueInclusion>(); }
 		}
 
 		public ValueInclusion[] PropertyUpdateGenerationInclusions
 		{
-			get { return new ValueInclusion[0]; }
+			get { return Array.Empty<ValueInclusion>(); }
 		}
 
 		public bool[] PropertyCheckability

src/NHibernate.Test/Async/Component/Basic/ComponentTest.cs

@@ -32,7 +32,7 @@ protected override string MappingsAssembly
 
 		protected override System.Collections.IList Mappings
 		{
-			get { return new string[] { }; }
+			get { return Array.Empty<string>(); }
 		}
 
 		protected override void Configure(Configuration configuration)

src/NHibernate.Test/Async/DialectTest/DialectFixture.cs

@@ -68,7 +68,7 @@ public async Task CurrentTimestampSelectionAsync()
 
 			using (var connection = await (sessions.ConnectionProvider.GetConnectionAsync(CancellationToken.None)))
 			{
-				var statement = driver.GenerateCommand(CommandType.Text, new SqlString(dialect.CurrentTimestampSelectString), new SqlType[0]);
+				var statement = driver.GenerateCommand(CommandType.Text, new SqlString(dialect.CurrentTimestampSelectString), Array.Empty<SqlType>());
 				statement.Connection = connection;
 				using (var reader = await (statement.ExecuteReaderAsync()))
 				{

src/NHibernate.Test/Async/Events/PostEvents/PostUpdateFixture.cs

@@ -8,6 +8,7 @@
 //------------------------------------------------------------------------------
 
 
+using System;
 using System.Collections;
 using System.Collections.Generic;
 using NHibernate.Event;
@@ -56,7 +57,7 @@ public async Task ImplicitFlushAsync()
 			}
 
 			await (DbCleanupAsync());
-			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = new IPostUpdateEventListener[0];
+			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = Array.Empty<IPostUpdateEventListener>();
 		}
 
 		[Test]
@@ -85,7 +86,7 @@ public async Task ExplicitUpdateAsync()
 			}
 
 			await (DbCleanupAsync());
-			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = new IPostUpdateEventListener[0];
+			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = Array.Empty<IPostUpdateEventListener>();
 		}
 
 		[Test]
@@ -123,7 +124,7 @@ public async Task WithDetachedObjectAsync()
 			}
 
 			await (DbCleanupAsync());
-			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = new IPostUpdateEventListener[0];
+			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = Array.Empty<IPostUpdateEventListener>();
 		}
 
 		[Test]
@@ -163,7 +164,7 @@ public async Task UpdateDetachedObjectAsync()
 			}
 
 			await (DbCleanupAsync());
-			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = new IPostUpdateEventListener[0];
+			((DebugSessionFactory) Sfi).EventListeners.PostUpdateEventListeners = Array.Empty<IPostUpdateEventListener>();
 		}
 
 		[Test]
@@ -202,7 +203,7 @@ public async Task UpdateDetachedObjectWithLockAsync()
 			}
 
 			await (DbCleanupAsync());
-			((DebugSessionFactory)Sfi).EventListeners.PostUpdateEventListeners = new IPostUpdateEventListener[0];
+			((DebugSessionFactory)Sfi).EventListeners.PostUpdateEventListeners = Array.Empty<IPostUpdateEventListener>();
 		}
 		private async Task DbCleanupAsync(CancellationToken cancellationToken = default(CancellationToken))
 		{

src/NHibernate.Test/Async/ExceptionsTest/NullQueryTest.cs

@@ -28,7 +28,7 @@ public class NullQueryTestAsync : TestCase
 
 		protected override IList Mappings
 		{
-			get { return new string[0]; }
+			get { return Array.Empty<string>(); }
 		}
 
 		#endregion

src/NHibernate.Test/Async/Legacy/FumTest.cs

@@ -86,7 +86,7 @@ public async Task CriteriaCollectionAsync()
 				Assert.IsTrue(b.MapComponent.Stringmap.Count == 2);
 
 				int none = (await (s.CreateCriteria(typeof(Fum))
-					.Add(Expression.In("FumString", new string[0]))
+					.Add(Expression.In("FumString", Array.Empty<string>()))
 					.ListAsync())).Count;
 				Assert.AreEqual(0, none);
 

src/NHibernate.Test/Async/LinqBulkManipulation/Fixture.cs

@@ -25,7 +25,7 @@ namespace NHibernate.Test.LinqBulkManipulation
 	[TestFixture]
 	public class FixtureAsync : TestCase
 	{
-		protected override IList Mappings => new string[0];
+		protected override IList Mappings => Array.Empty<string>();
 
 		protected override void Configure(Cfg.Configuration configuration)
 		{
@@ -970,7 +970,7 @@ public async Task SimpleDeleteOnAnimalAsync()
 			using (var t = s.BeginTransaction())
 			{
 				// Get rid of FK which may fail the test
-				_doll.Friends = new Human[0];
+				_doll.Friends = Array.Empty<Human>();
 				await (s.UpdateAsync(_doll));
 				await (t.CommitAsync());
 			}
@@ -1029,7 +1029,7 @@ public async Task DeleteOnJoinedSubclassAsync()
 			using (var t = s.BeginTransaction())
 			{
 				// Get rid of FK which may fail the test
-				_doll.Friends = new Human[0];
+				_doll.Friends = Array.Empty<Human>();
 				await (s.UpdateAsync(_doll));
 				await (t.CommitAsync());
 			}

src/NHibernate.Test/Async/MappingByCode/IntegrationTests/NH2825/FixtureByCode.cs

@@ -8,6 +8,7 @@
 //------------------------------------------------------------------------------
 
 
+using System;
 using System.Collections;
 using NHibernate.Cfg.MappingSchema;
 using NHibernate.Mapping.ByCode;
@@ -21,7 +22,7 @@ public class FixtureByCodeAsync : FixtureAsync
 	{
 		protected override IList Mappings
 		{
-			get { return new string[0]; }
+			get { return Array.Empty<string>(); }
 		}
 
 		protected override string MappingsAssembly
@@ -63,4 +64,4 @@ private HbmMapping GetMappings()
 			return mapper.CompileMappingForAllExplicitlyAddedEntities();
 		}
 	}
-}
+}