Should get a review from someone else before merging, since I made the last change.

Can you explain why you made that change? It looks like the PR was green.

It wasn't. That check is lying.

From jaredpar:
This is the first time MSBuild has used a compiler that has the new mtehod group to delegate caching feature. That changed the behavior of this line:

SendDataDelegate transportDelegate = PacketProcessor;

Prior to this version this was always a delegate allocation, now though it's a cached delegate conversion.
That means the transportDelegate is cached here and likely keeping what you expected to be collected in memory

Great info to put in the commit message . . .

I'm assuming it will be squashed anyway, so my commit message would disappear. You're right that it'll be good to include in the squash message.

I would not recommend squashing this, since it required an update to our code.

/azp run