Skip to content

Commit 2c6d2f7

Browse files
committed
Avoid double null-checking in OpenSession[Async]() calls
1 parent 29d716d commit 2c6d2f7

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

Orm/Xtensive.Orm/Orm/Domain.cs

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -234,14 +234,8 @@ public Session OpenSession(SessionType type)
234234
/// }
235235
/// </code></sample>
236236
/// <seealso cref="Session"/>
237-
public Session OpenSession(SessionConfiguration configuration)
238-
{
239-
ArgumentValidator.EnsureArgumentNotNull(configuration, "configuration");
240-
241-
return OpenSessionInternal(configuration,
242-
null,
243-
configuration.Supports(SessionOptions.AutoActivation));
244-
}
237+
public Session OpenSession(SessionConfiguration configuration) =>
238+
OpenSessionInternal(configuration, null, configuration.Supports(SessionOptions.AutoActivation));
245239

246240
internal Session OpenSessionInternal(SessionConfiguration configuration, StorageNode storageNode, bool activate)
247241
{
@@ -350,7 +344,6 @@ public Task<Session> OpenSessionAsync(SessionConfiguration configuration, Cancel
350344

351345
internal async Task<Session> OpenSessionInternalAsync(SessionConfiguration configuration, StorageNode storageNode, SessionScope sessionScope, CancellationToken cancellationToken)
352346
{
353-
ArgumentValidator.EnsureArgumentNotNull(configuration, nameof(configuration));
354347
configuration.Lock(true);
355348

356349
if (isDebugEventLoggingEnabled) {

0 commit comments

Comments
 (0)