GetAwaiter() Remarks Are Not Clear

[Yashuaa]

I've seen multiple discussions in the issues section about the remark below

This method is intended for compiler use rather than use directly in code.

For this specific method GetAwaiter() which is used like so:

var result = SomeAsyncMethod().GetAwaiter().GetResult();

So if devs shouldn't be using GetAwaiter() in code then we need to know, if there are 1 off situations when we should prefer GetAwaiter().GetResult() versus .Wait() - we need to know.

So the remark should either be removed or there should be clarification because at this point the stance from MSFT is "Don't use it" and that doesn't seem to be the case from @stephentoub 's comments.

I think the remark should just be removed altogether. It looked like someone tried to remove it but the PR got closed and never got merged.

It would be for here : https://learn.microsoft.com/en-us/dotnet/api/system.threading.tasks.task.getawaiter?view=net-7.0

[ghost]

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

[ghost]

Tagging subscribers to this area: @dotnet/area-system-threading-tasks See info in area-owners.md if you want to be subscribed.

Issue Details

---

I've seen multiple discussions in the issues section about the remark below

This method is intended for compiler use rather than use directly in code.

For this specific method GetAwaiter() which is used like so:

var result = SomeAsyncMethod().GetAwaiter().GetResult();

So if devs shouldn't be using GetAwaiter() in code then we need to know, if there are 1 off situations when we should prefer GetAwaiter().GetResult() versus .Wait() - we need to know.

So the remark should either be removed or there should be clarification because at this point the stance from MSFT is "Don't use it" and that doesn't seem to be the case from @stephentoub 's comments.

I think the remark should just be removed altogether. It looked like someone tried to remove it but the PR got closed and never got merged.

It would be for here : https://learn.microsoft.com/en-us/dotnet/api/system.threading.tasks.task.getawaiter?view=net-7.0

Author:	Yashuaa
Assignees:	-
Labels:	untriaged, Pri3, area-System.Threading.Tasks, needs-area-label
Milestone:	-

[stephentoub]

We should delete those comments from the awaiters from Task and Task<TResult>. They're no longer accurate.

[Yashuaa]

Did anyone ever get on this ?