Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Avoid a closure allocation in MessageProcessingHandler #7788

Merged
davidsh merged 5 commits into from
Apr 23, 2016
Merged

Avoid a closure allocation in MessageProcessingHandler #7788

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
7bccae8
Avoid a closure allocation in MessageProcessingHandler
jamesqo Apr 16, 2016
6eb94a0
Respond to PR feedback: Introduce SendState private class
jamesqo Apr 16, 2016
980d674
Respond to PR feedback, #2
jamesqo Apr 16, 2016
2e4788b
public -> internal in SendState
jamesqo Apr 20, 2016
1790fd4
SendState -> var
jamesqo Apr 22, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
public -> internal in SendState
  • Loading branch information
@jamesqo
jamesqo committed Apr 20, 2016
commit 2e4788b3cc4f37f8edfd9827ccfbff5c76e79453
4 changes: 2 additions & 2 deletions src/System.Net.Http/src/System/Net/Http/MessageProcessingHandler.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,8 +126,8 @@ private static void HandleCanceledOperations(CancellationToken cancellationToken
// a closure, while simultaneously avoiding a tuple allocation.
private sealed class SendState : TaskCompletionSource<HttpResponseMessage>
{
public readonly MessageProcessingHandler _handler;
public readonly CancellationToken _token;
internal readonly MessageProcessingHandler _handler;
internal readonly CancellationToken _token;

public SendState(MessageProcessingHandler handler, CancellationToken token)
{
Expand Down