Reduce allocation in query for execution strategy #24208
Conversation
|
|
||
| return result; | ||
| return false; |
There was a problem hiding this comment.
Why was it true before? 
Doesn't matter, it's not used :) I can make it true if we care.
There was a problem hiding this comment.
Why was it true before?
Don't know.
I am a bot, I question for changes only disregarding correctness of old code. 😉
Can use whatever if the value is irrelevant. Curiosity why not void if value is not used.
There was a problem hiding this comment.
Only because Execute/ExecuteAsync return TResult, and C# doesn't allow void.
There was a problem hiding this comment.
We could have overloads which don't return TResult, but...
|
Hello @roji! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
|
Maybe this? https://marketplace.visualstudio.com/items?itemName=MukulSabharwal.ClrHeapAllocationAnalyzer For Resharper/Rider, there's the great Heap Allocations Viewer which underlines all allocations. |
|
I basically just get very, very alert when I'm around lambdas. |
No description provided.