Skip to content

Conversation

danielcweber
Copy link
Collaborator

@danielcweber danielcweber commented Jun 11, 2018

Save allocation of lots of closures and allow delegate caching!

@danielcweber danielcweber changed the title Use the stateful overload of AsyncLock in some places to save allocation of closures and allow delegate caching. Use the stateful overload of AsyncLock in some places. Jun 11, 2018
@@ -155,7 +155,7 @@ protected override void Dispose(bool disposing)
if (disposing)
{
Disposable.TryDispose(ref _timerSerial);
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misaligned }

@@ -305,7 +305,7 @@ protected override void Dispose(bool disposing)
if (disposing)
{
Disposable.TryDispose(ref _periodicDisposable);
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misaligned }

@@ -399,7 +399,7 @@ protected override void Dispose(bool disposing)
if (disposing)
{
Disposable.TryDispose(ref _timerSerial);
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misaligned }

}

protected override void Dispose(bool disposing)
{
if (disposing)
{
Disposable.TryDispose(ref _bufferClosingSerialDisposable);
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misaligned }

@@ -663,7 +663,7 @@ protected override void Dispose(bool disposing)
if (disposing)
{
Disposable.TryDispose(ref _boundariesDisposable);
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misaligned }

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unclear how that happened....fixed.

@danielcweber danielcweber merged commit 7551d1a into dotnet:master Jun 12, 2018
@danielcweber danielcweber deleted the UseAsyncLockWithState branch June 12, 2018 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants