Reuse the same generic EmptyMapClass instance across the project (#1477)

bahusoid · fredericDelaporte · commit c4eb6f4c40f5 · 2017-12-10T12:31:41.000+01:00
diff --git a/src/NHibernate.Test/Async/Hql/Ast/HqlFixture.cs b/src/NHibernate.Test/Async/Hql/Ast/HqlFixture.cs
@@ -33,7 +33,7 @@ protected override bool AppliesTo(Dialect.Dialect dialect)
 
 		protected HQLQueryPlan CreateQueryPlan(string hql, bool scalar)
 		{
-			return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);
+			return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);
 		}
 
 		protected HQLQueryPlan CreateQueryPlan(string hql)
diff --git a/src/NHibernate.Test/BulkManipulation/BaseFixture.cs b/src/NHibernate.Test/BulkManipulation/BaseFixture.cs
@@ -7,7 +7,7 @@ namespace NHibernate.Test.BulkManipulation
 {
 	public class BaseFixture: TestCase
 	{
-		private readonly IDictionary<string, IFilter> emptyfilters = new CollectionHelper.EmptyMapClass<string, IFilter>();
+		private readonly IDictionary<string, IFilter> emptyfilters = CollectionHelper.EmptyDictionary<string, IFilter>();
 
 		#region Overrides of TestCase
 
diff --git a/src/NHibernate.Test/Hql/Ast/BaseFixture.cs b/src/NHibernate.Test/Hql/Ast/BaseFixture.cs
@@ -7,7 +7,7 @@ namespace NHibernate.Test.Hql.Ast
 {
 	public class BaseFixture: TestCase
 	{
-		private readonly IDictionary<string, IFilter> emptyfilters = new CollectionHelper.EmptyMapClass<string, IFilter>();
+		private readonly IDictionary<string, IFilter> emptyfilters = CollectionHelper.EmptyDictionary<string, IFilter>();
 		
 		#region Overrides of TestCase
 
diff --git a/src/NHibernate.Test/Hql/Ast/HqlFixture.cs b/src/NHibernate.Test/Hql/Ast/HqlFixture.cs
@@ -22,7 +22,7 @@ protected override bool AppliesTo(Dialect.Dialect dialect)
 
 		protected HQLQueryPlan CreateQueryPlan(string hql, bool scalar)
 		{
-			return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);
+			return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);
 		}
 
 		protected HQLQueryPlan CreateQueryPlan(string hql)
diff --git a/src/NHibernate.Test/NHSpecificTest/NH1849/Fixture.cs b/src/NHibernate.Test/NHSpecificTest/NH1849/Fixture.cs
@@ -46,7 +46,7 @@ public void ExecutesCustomSqlFunctionContains()
 		{
 		 string hql = @"from Customer c where contains(c.Name, :smth)";
 
-		 HQLQueryPlan plan = new QueryExpressionPlan(new StringQueryExpression(hql), false, new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);
+		 HQLQueryPlan plan = new QueryExpressionPlan(new StringQueryExpression(hql), false, CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);
 
 		 Assert.AreEqual(1, plan.ParameterMetadata.NamedParameterNames.Count);
 		 Assert.AreEqual(1, plan.QuerySpaces.Count);
diff --git a/src/NHibernate.Test/NHSpecificTest/NH2031/HqlModFuctionForMsSqlTest.cs b/src/NHibernate.Test/NHSpecificTest/NH2031/HqlModFuctionForMsSqlTest.cs
@@ -28,7 +28,7 @@ public void TheModuleOperationShouldAddParenthesisToAvoidWrongSentence()
 
 		public string GetSql(string query)
 		{
-			var qt = new QueryTranslatorImpl(null, new HqlParseEngine(query, false, Sfi).Parse(), new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);
+			var qt = new QueryTranslatorImpl(null, new HqlParseEngine(query, false, Sfi).Parse(), CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);
 			qt.Compile(null, false);
 			return qt.SQLString;
 		}
diff --git a/src/NHibernate/Cfg/MappingSchema/AbstractDecoratable.cs b/src/NHibernate/Cfg/MappingSchema/AbstractDecoratable.cs
@@ -9,7 +9,7 @@ namespace NHibernate.Cfg.MappingSchema
 	[Serializable]
 	public abstract class AbstractDecoratable : IDecoratable
 	{
-		private static readonly IDictionary<string, MetaAttribute> EmptyMetaData = new CollectionHelper.EmptyMapClass<string, MetaAttribute>();
+		private static readonly IDictionary<string, MetaAttribute> EmptyMetaData = CollectionHelper.EmptyDictionary<string, MetaAttribute>();
 
 		[NonSerialized]
 		[XmlIgnore]
diff --git a/src/NHibernate/Cfg/XmlHbmBinding/Binder.cs b/src/NHibernate/Cfg/XmlHbmBinding/Binder.cs
@@ -13,7 +13,7 @@ public abstract class Binder
 		protected static readonly INHibernateLogger log = NHibernateLogger.For(typeof (Binder));
 
 		protected static readonly IDictionary<string, MetaAttribute> EmptyMeta =
-			new CollectionHelper.EmptyMapClass<string, MetaAttribute>();
+			CollectionHelper.EmptyDictionary<string, MetaAttribute>();
 
 		protected readonly Mappings mappings;
 
diff --git a/src/NHibernate/Cfg/XmlHbmBinding/ResultSetMappingBinder.cs b/src/NHibernate/Cfg/XmlHbmBinding/ResultSetMappingBinder.cs
@@ -309,7 +309,7 @@ private IDictionary<string, string[]> BindPropertyResults(string alias, HbmRetur
 			{
 				newPropertyResults[entry.Key] = entry.Value;
 			}
-			return newPropertyResults.Count == 0 ? (IDictionary<string, string[]>)new CollectionHelper.EmptyMapClass<string, string[]>() : newPropertyResults;
+			return newPropertyResults.Count == 0 ? (IDictionary<string, string[]>)CollectionHelper.EmptyDictionary<string, string[]>() : newPropertyResults;
 		}
 
 		private static List<string> GetResultColumns(HbmReturnProperty returnPropertySchema)
diff --git a/src/NHibernate/Engine/JoinSequence.cs b/src/NHibernate/Engine/JoinSequence.cs
@@ -128,7 +128,7 @@ public JoinSequence AddJoin(IAssociationType associationType, string alias, Join
 
 		public JoinFragment ToJoinFragment()
 		{
-			return ToJoinFragment(new CollectionHelper.EmptyMapClass<string, IFilter>(), true);
+			return ToJoinFragment(CollectionHelper.EmptyDictionary<string, IFilter>(), true);
 		}
 
 		public JoinFragment ToJoinFragment(IDictionary<string, IFilter> enabledFilters, bool includeExtraJoins)
diff --git a/src/NHibernate/Hql/Ast/ANTLR/HqlSqlWalker.cs b/src/NHibernate/Hql/Ast/ANTLR/HqlSqlWalker.cs
@@ -393,7 +393,7 @@ private void PostProcessDML(IRestrictableStatement statement)
 			if (persister.DiscriminatorType != null)
 			{
 				new SyntheticAndFactory(this)
-					.AddDiscriminatorWhereFragment(statement, persister, new CollectionHelper.EmptyMapClass<string, IFilter>(), fromElement.TableAlias);
+					.AddDiscriminatorWhereFragment(statement, persister, CollectionHelper.EmptyDictionary<string, IFilter>(), fromElement.TableAlias);
 			}
 		}
 
diff --git a/src/NHibernate/Impl/SessionFactoryImpl.cs b/src/NHibernate/Impl/SessionFactoryImpl.cs
@@ -616,15 +616,15 @@ public string GetIdentifierPropertyName(string className)
 		public IType[] GetReturnTypes(String queryString)
 		{
 			return
-				queryPlanCache.GetHQLQueryPlan(queryString.ToQueryExpression(), false, new CollectionHelper.EmptyMapClass<string, IFilter>()).
+				queryPlanCache.GetHQLQueryPlan(queryString.ToQueryExpression(), false, CollectionHelper.EmptyDictionary<string, IFilter>()).
 					ReturnMetadata.ReturnTypes;
 		}
 
 		/// <summary> Get the return aliases of a query</summary>
 		public string[] GetReturnAliases(string queryString)
 		{
 			return
-				queryPlanCache.GetHQLQueryPlan(queryString.ToQueryExpression(), false, new CollectionHelper.EmptyMapClass<string, IFilter>()).
+				queryPlanCache.GetHQLQueryPlan(queryString.ToQueryExpression(), false, CollectionHelper.EmptyDictionary<string, IFilter>()).
 					ReturnMetadata.ReturnAliases;
 		}
 
@@ -1163,7 +1163,7 @@ private IDictionary<string, HibernateException> CheckNamedQueries()
 				{
 					log.Debug("Checking named query: {0}", queryName);
 					//TODO: BUG! this currently fails for named queries for non-POJO entities
-					queryPlanCache.GetHQLQueryPlan(qd.QueryString.ToQueryExpression(), false, new CollectionHelper.EmptyMapClass<string, IFilter>());
+					queryPlanCache.GetHQLQueryPlan(qd.QueryString.ToQueryExpression(), false, CollectionHelper.EmptyDictionary<string, IFilter>());
 				}
 				catch (QueryException e)
 				{
diff --git a/src/NHibernate/Impl/SqlQueryImpl.cs b/src/NHibernate/Impl/SqlQueryImpl.cs
@@ -93,7 +93,7 @@ protected internal override IDictionary<string, LockMode> LockModes
 			get
 			{
 				//we never need to apply locks to the SQL
-				return new CollectionHelper.EmptyMapClass<string, LockMode>();
+				return CollectionHelper.EmptyDictionary<string, LockMode>();
 			}
 		}
 
@@ -236,7 +236,7 @@ public ISQLQuery AddJoin(string alias, string path, LockMode lockMode)
 			string ownerAlias = path.Substring(0, loc);
 			string role = path.Substring(loc + 1);
 			queryReturns.Add(
-				new NativeSQLQueryJoinReturn(alias, ownerAlias, role, new CollectionHelper.EmptyMapClass<string, string[]>(), lockMode));
+				new NativeSQLQueryJoinReturn(alias, ownerAlias, role, CollectionHelper.EmptyDictionary<string, string[]>(), lockMode));
 			return this;
 		}
 
diff --git a/src/NHibernate/Impl/StatelessSessionImpl.cs b/src/NHibernate/Impl/StatelessSessionImpl.cs
@@ -288,7 +288,7 @@ public override IType GetFilterParameterType(string filterParameterName)
 
 		public override IDictionary<string, IFilter> EnabledFilters
 		{
-			get { return new CollectionHelper.EmptyMapClass<string, IFilter>(); }
+			get { return CollectionHelper.EmptyDictionary<string, IFilter>(); }
 		}
 
 		public override IQueryTranslator[] GetQueries(IQueryExpression query, bool scalar)
diff --git a/src/NHibernate/Loader/AbstractEntityJoinWalker.cs b/src/NHibernate/Loader/AbstractEntityJoinWalker.cs
@@ -33,7 +33,7 @@ protected virtual void InitAll(SqlString whereString, SqlString orderByString, L
 			IList<OuterJoinableAssociation> allAssociations = new List<OuterJoinableAssociation>(associations);
 			allAssociations.Add(
 				new OuterJoinableAssociation(persister.EntityType, null, null, alias, JoinType.LeftOuterJoin, null, Factory,
-											 new CollectionHelper.EmptyMapClass<string, IFilter>()));
+											 CollectionHelper.EmptyDictionary<string, IFilter>()));
 
 			InitPersisters(allAssociations, lockMode);
 			InitStatementString(whereString, orderByString, lockMode);
diff --git a/src/NHibernate/Loader/Collection/OneToManyJoinWalker.cs b/src/NHibernate/Loader/Collection/OneToManyJoinWalker.cs
@@ -38,7 +38,7 @@ public OneToManyJoinWalker(IQueryableCollection oneToManyPersister, int batchSiz
 			IList<OuterJoinableAssociation> allAssociations = new List<OuterJoinableAssociation>(associations);
 			allAssociations.Add(
 				new OuterJoinableAssociation(oneToManyPersister.CollectionType, null, null, alias, JoinType.LeftOuterJoin, null, Factory,
-				                             new CollectionHelper.EmptyMapClass<string, IFilter>()));
+				                             CollectionHelper.EmptyDictionary<string, IFilter>()));
 
 			InitPersisters(allAssociations, LockMode.None);
 			InitStatementString(elementPersister, alias, batchSize, subquery);
diff --git a/src/NHibernate/Loader/DefaultEntityAliases.cs b/src/NHibernate/Loader/DefaultEntityAliases.cs
@@ -19,7 +19,7 @@ public class DefaultEntityAliases : IEntityAliases
 		private readonly IDictionary<string, string[]> userProvidedAliases;
 
 		public DefaultEntityAliases(ILoadable persister, string suffix)
-			: this(new CollectionHelper.EmptyMapClass<string, string[]>(), persister, suffix) {}
+			: this(CollectionHelper.EmptyDictionary<string, string[]>(), persister, suffix) {}
 
 		/// <summary>
 		/// Calculate and cache select-clause suffixes.
@@ -147,4 +147,4 @@ private static void Intern(string[] strings)
 			}
 		}
 	}
-}
+}
diff --git a/src/NHibernate/Loader/Entity/CascadeEntityJoinWalker.cs b/src/NHibernate/Loader/Entity/CascadeEntityJoinWalker.cs
@@ -12,12 +12,12 @@ public class CascadeEntityJoinWalker : AbstractEntityJoinWalker
 
 		public CascadeEntityJoinWalker(IOuterJoinLoadable persister, CascadingAction action,
 		                               ISessionFactoryImplementor factory)
-			: base(persister, factory, new CollectionHelper.EmptyMapClass<string, IFilter>())
+			: base(persister, factory, CollectionHelper.EmptyDictionary<string, IFilter>())
 		{
 			cascadeAction = action;
 			SqlStringBuilder whereCondition = WhereString(Alias, persister.IdentifierColumnNames, 1)
 				//include the discriminator and class-level where, but not filters
-				.Add(persister.FilterFragment(Alias, new CollectionHelper.EmptyMapClass<string, IFilter>()));
+				.Add(persister.FilterFragment(Alias, CollectionHelper.EmptyDictionary<string, IFilter>()));
 
 			InitAll(whereCondition.ToSqlString(), SqlString.Empty, LockMode.Read);
 		}
diff --git a/src/NHibernate/Loader/Entity/CascadeEntityLoader.cs b/src/NHibernate/Loader/Entity/CascadeEntityLoader.cs
@@ -7,7 +7,7 @@ namespace NHibernate.Loader.Entity
 	public class CascadeEntityLoader : AbstractEntityLoader
 	{
 		public CascadeEntityLoader(IOuterJoinLoadable persister, CascadingAction action, ISessionFactoryImplementor factory)
-			: base(persister, persister.IdentifierType, factory, new CollectionHelper.EmptyMapClass<string, IFilter>())
+			: base(persister, persister.IdentifierType, factory, CollectionHelper.EmptyDictionary<string, IFilter>())
 		{
 			JoinWalker walker = new CascadeEntityJoinWalker(persister, action, factory);
 			InitFromWalker(walker);
diff --git a/src/NHibernate/Loader/Entity/EntityJoinWalker.cs b/src/NHibernate/Loader/Entity/EntityJoinWalker.cs
@@ -23,7 +23,7 @@ public EntityJoinWalker(IOuterJoinLoadable persister, string[] uniqueKey, int ba
 
 			SqlStringBuilder whereCondition = WhereString(Alias, uniqueKey, batchSize)
 				//include the discriminator and class-level where, but not filters
-				.Add(persister.FilterFragment(Alias, new CollectionHelper.EmptyMapClass<string, IFilter>()));
+				.Add(persister.FilterFragment(Alias, CollectionHelper.EmptyDictionary<string, IFilter>()));
 
 			InitAll(whereCondition.ToSqlString(), SqlString.Empty, lockMode);
 		}
diff --git a/src/NHibernate/Loader/GeneratedCollectionAliases.cs b/src/NHibernate/Loader/GeneratedCollectionAliases.cs
@@ -37,7 +37,7 @@ public GeneratedCollectionAliases(IDictionary<string, string[]> userProvidedAlia
 		}
 
 		public GeneratedCollectionAliases(ICollectionPersister persister, string str)
-			: this(new CollectionHelper.EmptyMapClass<string, string[]>(), persister, str) {}
+			: this(CollectionHelper.EmptyDictionary<string, string[]>(), persister, str) {}
 
 		private string[] GetUserProvidedCompositeElementAliases(string[] defaultAliases)
 		{
diff --git a/src/NHibernate/Persister/Collection/AbstractCollectionPersister.cs b/src/NHibernate/Persister/Collection/AbstractCollectionPersister.cs
@@ -559,7 +559,7 @@ public AbstractCollectionPersister(Mapping.Collection collection, ICacheConcurre
 		public void PostInstantiate()
 		{
 			initializer = queryLoaderName == null
-							? CreateCollectionInitializer(new CollectionHelper.EmptyMapClass<string, IFilter>())
+							? CreateCollectionInitializer(CollectionHelper.EmptyDictionary<string, IFilter>())
 							: new NamedQueryCollectionInitializer(queryLoaderName, this);
 		}
 
diff --git a/src/NHibernate/Persister/Entity/AbstractEntityPersister.cs b/src/NHibernate/Persister/Entity/AbstractEntityPersister.cs
@@ -2118,7 +2118,7 @@ protected void CreateUniqueKeyLoaders()
 					//don't need filters for the static loaders
 					uniqueKeyLoaders[propertyNames[i]] =
 						CreateUniqueKeyLoader(propertyTypes[i], GetPropertyColumnNames(i),
-																	new CollectionHelper.EmptyMapClass<string, IFilter>());
+																	CollectionHelper.EmptyDictionary<string, IFilter>());
 				}
 			}
 		}
@@ -2197,7 +2197,7 @@ protected IUniqueEntityLoader CreateEntityLoader(LockMode lockMode, IDictionary<
 
 		protected IUniqueEntityLoader CreateEntityLoader(LockMode lockMode)
 		{
-			return CreateEntityLoader(lockMode, new CollectionHelper.EmptyMapClass<string, IFilter>());
+			return CreateEntityLoader(lockMode, CollectionHelper.EmptyDictionary<string, IFilter>());
 		}
 
 		protected bool Check(int rows, object id, int tableNumber, IExpectation expectation, DbCommand statement)
diff --git a/src/NHibernate/Util/CollectionHelper.cs b/src/NHibernate/Util/CollectionHelper.cs
@@ -214,6 +214,12 @@ public IEnumerator GetEnumerator()
 
 		public static readonly IEnumerable EmptyEnumerable = new EmptyEnumerableClass();
 		public static readonly IDictionary EmptyMap = new EmptyMapClass();
+
+		public static IDictionary<TKey, TValue> EmptyDictionary<TKey, TValue>()
+		{
+			return EmptyMapClass<TKey, TValue>.Instance;
+		}
+
 		public static readonly ICollection EmptyCollection = EmptyMap;
 		// Since v5
 		[Obsolete("It has no more usages in NHibernate and will be removed in a future version.")]
@@ -463,8 +469,18 @@ public object Current
 		[Serializable]
 		public class EmptyMapClass<TKey, TValue> : IDictionary<TKey, TValue>
 		{
+#pragma warning disable 618 // Constructor is obsolete, to be switched to non-obsolete but private.
+			internal static readonly IDictionary<TKey, TValue> Instance = new EmptyMapClass<TKey, TValue>();
+#pragma warning restore 618
+
 			private static readonly EmptyEnumerator<TKey, TValue> emptyEnumerator = new EmptyEnumerator<TKey, TValue>();
 
+			// Since v5.1. To be switched to private.
+			[Obsolete("Please use CollectionHelper.EmptyDictionary<TKey, TValue>() instead.")]
+			public EmptyMapClass()
+			{
+			}
+
 			#region IDictionary<TKey,TValue> Members
 
 			public bool ContainsKey(TKey key)

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ protected override bool AppliesTo(Dialect.Dialect dialect)`
`33`	`33`
`34`	`34`	`protected HQLQueryPlan CreateQueryPlan(string hql, bool scalar)`
`35`	`35`	`{`
`36`		`- return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);`
	`36`	`+ return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`protected HQLQueryPlan CreateQueryPlan(string hql)`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ namespace NHibernate.Test.BulkManipulation`
`7`	`7`	`{`
`8`	`8`	`public class BaseFixture: TestCase`
`9`	`9`	`{`
`10`		`- private readonly IDictionary<string, IFilter> emptyfilters = new CollectionHelper.EmptyMapClass<string, IFilter>();`
	`10`	`+ private readonly IDictionary<string, IFilter> emptyfilters = CollectionHelper.EmptyDictionary<string, IFilter>();`
`11`	`11`
`12`	`12`	`#region Overrides of TestCase`
`13`	`13`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ protected override bool AppliesTo(Dialect.Dialect dialect)`
`22`	`22`
`23`	`23`	`protected HQLQueryPlan CreateQueryPlan(string hql, bool scalar)`
`24`	`24`	`{`
`25`		`- return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);`
	`25`	`+ return new QueryExpressionPlan(new StringQueryExpression(hql), scalar, CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`protected HQLQueryPlan CreateQueryPlan(string hql)`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ public void ExecutesCustomSqlFunctionContains()`
`46`	`46`	`{`
`47`	`47`	`string hql = @"from Customer c where contains(c.Name, :smth)";`
`48`	`48`
`49`		`- HQLQueryPlan plan = new QueryExpressionPlan(new StringQueryExpression(hql), false, new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);`
	`49`	`+ HQLQueryPlan plan = new QueryExpressionPlan(new StringQueryExpression(hql), false, CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);`
`50`	`50`
`51`	`51`	`Assert.AreEqual(1, plan.ParameterMetadata.NamedParameterNames.Count);`
`52`	`52`	`Assert.AreEqual(1, plan.QuerySpaces.Count);`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public void TheModuleOperationShouldAddParenthesisToAvoidWrongSentence()`
`28`	`28`
`29`	`29`	`public string GetSql(string query)`
`30`	`30`	`{`
`31`		`- var qt = new QueryTranslatorImpl(null, new HqlParseEngine(query, false, Sfi).Parse(), new CollectionHelper.EmptyMapClass<string, IFilter>(), Sfi);`
	`31`	`+ var qt = new QueryTranslatorImpl(null, new HqlParseEngine(query, false, Sfi).Parse(), CollectionHelper.EmptyDictionary<string, IFilter>(), Sfi);`
`32`	`32`	`qt.Compile(null, false);`
`33`	`33`	`return qt.SQLString;`
`34`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ namespace NHibernate.Cfg.MappingSchema`
`9`	`9`	`[Serializable]`
`10`	`10`	`public abstract class AbstractDecoratable : IDecoratable`
`11`	`11`	`{`
`12`		`- private static readonly IDictionary<string, MetaAttribute> EmptyMetaData = new CollectionHelper.EmptyMapClass<string, MetaAttribute>();`
	`12`	`+ private static readonly IDictionary<string, MetaAttribute> EmptyMetaData = CollectionHelper.EmptyDictionary<string, MetaAttribute>();`
`13`	`13`
`14`	`14`	`[NonSerialized]`
`15`	`15`	`[XmlIgnore]`
Original file line number	Diff line number	Diff line change
`@@ -309,7 +309,7 @@ private IDictionary<string, string[]> BindPropertyResults(string alias, HbmRetur`
`309`	`309`	`{`
`310`	`310`	`newPropertyResults[entry.Key] = entry.Value;`
`311`	`311`	`}`
`312`		`- return newPropertyResults.Count == 0 ? (IDictionary<string, string[]>)new CollectionHelper.EmptyMapClass<string, string[]>() : newPropertyResults;`
	`312`	`+ return newPropertyResults.Count == 0 ? (IDictionary<string, string[]>)CollectionHelper.EmptyDictionary<string, string[]>() : newPropertyResults;`
`313`	`313`	`}`
`314`	`314`
`315`	`315`	`private static List<string> GetResultColumns(HbmReturnProperty returnPropertySchema)`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ public JoinSequence AddJoin(IAssociationType associationType, string alias, Join`
`128`	`128`
`129`	`129`	`public JoinFragment ToJoinFragment()`
`130`	`130`	`{`
`131`		`- return ToJoinFragment(new CollectionHelper.EmptyMapClass<string, IFilter>(), true);`
	`131`	`+ return ToJoinFragment(CollectionHelper.EmptyDictionary<string, IFilter>(), true);`
`132`	`132`	`}`
`133`	`133`
`134`	`134`	`public JoinFragment ToJoinFragment(IDictionary<string, IFilter> enabledFilters, bool includeExtraJoins)`