Skip to content

Commit 8642b19

Browse files
committed
Use collection types for private members
1 parent 7997e22 commit 8642b19

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

44 files changed

+97
-99
lines changed

src/NHibernate/AdoNet/AbstractBatcher.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public abstract partial class AbstractBatcher : IBatcher
3737
private SqlType[] _batchCommandParameterTypes;
3838
private readonly HashSet<DbCommand> _commandsToClose = new HashSet<DbCommand>();
3939
private readonly HashSet<DbDataReader> _readersToClose = new HashSet<DbDataReader>();
40-
private readonly IDictionary<DbDataReader, Stopwatch> _readersDuration = new Dictionary<DbDataReader, Stopwatch>();
40+
private readonly Dictionary<DbDataReader, Stopwatch> _readersDuration = new Dictionary<DbDataReader, Stopwatch>();
4141
private DbCommand _lastQuery;
4242
private bool _releasing;
4343

src/NHibernate/AdoNet/HanaBatchingBatcher.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public partial class HanaBatchingBatcher : AbstractBatcher
1717
private int _countOfCommands;
1818
private int _totalExpectedRowsAffected;
1919
private DbCommand _currentBatch;
20-
private readonly IList<DbCommand> _currentBatchCommands = new List<DbCommand>();
20+
private readonly List<DbCommand> _currentBatchCommands = new List<DbCommand>();
2121
private StringBuilder _currentBatchCommandsLog;
2222

2323
public HanaBatchingBatcher(ConnectionManager connectionManager, IInterceptor interceptor)

src/NHibernate/AdoNet/OracleDataClientBatchingBatcher.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ public partial class OracleDataClientBatchingBatcher : AbstractBatcher
1717
private int _countOfCommands;
1818
private int _totalExpectedRowsAffected;
1919
private DbCommand _currentBatch;
20-
private IDictionary<string, List<object>> _parameterValueListHashTable;
21-
private IDictionary<string, bool> _parameterIsAllNullsHashTable;
20+
private Dictionary<string, List<object>> _parameterValueListHashTable;
21+
private Dictionary<string, bool> _parameterIsAllNullsHashTable;
2222
private StringBuilder _currentBatchCommandsLog;
2323

2424
public OracleDataClientBatchingBatcher(ConnectionManager connectionManager, IInterceptor interceptor)

src/NHibernate/Async/Collection/Generic/PersistentGenericSet.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,4 +169,4 @@ public override Task<bool> NeedsUpdatingAsync(object entry, int i, IType elemTyp
169169
}
170170
}
171171
}
172-
}
172+
}

src/NHibernate/Cfg/Configuration.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -818,7 +818,7 @@ public Configuration AddAssembly(Assembly assembly)
818818
return this;
819819
}
820820

821-
private static IList<string> GetAllHbmXmlResourceNames(Assembly assembly)
821+
private static List<string> GetAllHbmXmlResourceNames(Assembly assembly)
822822
{
823823
var result = new List<string>();
824824

src/NHibernate/Cfg/ConfigurationSchema/EventConfiguration.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public ListenerType Type
4949
get { return type; }
5050
}
5151

52-
private IList<ListenerConfiguration> listeners = new List<ListenerConfiguration>();
52+
private List<ListenerConfiguration> listeners = new List<ListenerConfiguration>();
5353
/// <summary>
5454
/// Listeners for this event.
5555
/// </summary>

src/NHibernate/Cfg/MappingSchema/HbmDatabaseObject.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public HbmDefinition FindDefinition()
1717

1818
public IList<string> FindDialectScopeNames()
1919
{
20-
IList<string> dialectScopeNames = new List<string>();
20+
var dialectScopeNames = new List<string>();
2121

2222
if (dialectscope != null)
2323
foreach (HbmDialectScope dialectScopeSchema in dialectscope)
@@ -37,4 +37,4 @@ public bool HasDefinition()
3737
return FindDefinition() != null;
3838
}
3939
}
40-
}
40+
}

src/NHibernate/Cfg/MappingsQueue.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ namespace NHibernate.Cfg
1111
public class MappingsQueue
1212
{
1313
private readonly Queue availableEntries = new Queue();
14-
private readonly ISet<string> processedClassNames = new HashSet<string>();
14+
private readonly HashSet<string> processedClassNames = new HashSet<string>();
1515

1616
private readonly List<MappingsQueueEntry> unavailableEntries = new List<MappingsQueueEntry>();
1717

@@ -109,4 +109,4 @@ private static string FormatExceptionMessage(IEnumerable<MappingsQueueEntry> res
109109
return message.ToString();
110110
}
111111
}
112-
}
112+
}

src/NHibernate/Cfg/SessionFactoryConfigurationBase.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,12 @@ namespace NHibernate.Cfg
66
public class SessionFactoryConfigurationBase : ISessionFactoryConfiguration
77
{
88
private string name = string.Empty;
9-
private readonly IDictionary<string, string> properties = new Dictionary<string, string>();
10-
private readonly IList<MappingConfiguration> mappings = new List<MappingConfiguration>();
11-
private readonly IList<ClassCacheConfiguration> classesCache= new List<ClassCacheConfiguration>();
12-
private readonly IList<CollectionCacheConfiguration> collectionsCache= new List<CollectionCacheConfiguration>();
13-
private readonly IList<EventConfiguration> events= new List<EventConfiguration>();
14-
private readonly IList<ListenerConfiguration> listeners= new List<ListenerConfiguration>();
9+
private readonly Dictionary<string, string> properties = new Dictionary<string, string>();
10+
private readonly List<MappingConfiguration> mappings = new List<MappingConfiguration>();
11+
private readonly List<ClassCacheConfiguration> classesCache= new List<ClassCacheConfiguration>();
12+
private readonly List<CollectionCacheConfiguration> collectionsCache= new List<CollectionCacheConfiguration>();
13+
private readonly List<EventConfiguration> events= new List<EventConfiguration>();
14+
private readonly List<ListenerConfiguration> listeners= new List<ListenerConfiguration>();
1515

1616
/// <summary>
1717
/// The session factory name.
@@ -70,4 +70,4 @@ public IList<ListenerConfiguration> Listeners
7070
get { return listeners; }
7171
}
7272
}
73-
}
73+
}

src/NHibernate/Cfg/XmlHbmBinding/NamedSQLQueryBinder.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ public void AddSqlQuery(HbmSqlQuery querySchema)
3333
? querySchema.cachemode.ToCacheMode()
3434
: null;
3535

36-
IDictionary<string,string> parameterTypes = new LinkedHashMap<string,string>();
37-
IList<string> synchronizedTables = GetSynchronizedTables(querySchema);
36+
var parameterTypes = new LinkedHashMap<string,string>();
37+
var synchronizedTables = GetSynchronizedTables(querySchema);
3838

3939
NamedSQLQueryDefinition namedQuery;
4040

@@ -58,9 +58,9 @@ public void AddSqlQuery(HbmSqlQuery querySchema)
5858
});
5959
}
6060

61-
private static IList<string> GetSynchronizedTables(HbmSqlQuery querySchema)
61+
private static List<string> GetSynchronizedTables(HbmSqlQuery querySchema)
6262
{
63-
IList<string> synchronizedTables = new List<string>();
63+
var synchronizedTables = new List<string>();
6464

6565
foreach (object item in querySchema.Items ?? Array.Empty<object>())
6666
{

0 commit comments

Comments
 (0)