Clarify behavior of OfType() with null values

[matthiaslischka]

OfType<T> does not take all elements that can be cast. It skips null.
And consequently the hint that Cast<T> would instead throw is a bit misleading too.

new List<string> {"Mango", null, "Orange"}.OfType<string>();
// --> ["Mango", "Orange"]
	
new List<string> {"Mango", null, "Orange"}.Cast<string>();
// --> ["Mango", null, "Orange"]

BR Matthias

[opbld33]

Docs Build status updates of commit 0ce71d4:

✅ Validation status: passed

File	Status	Preview URL	Details
samples/snippets/csharp/VS_Snippets_CLR_System/system.Linq.Enumerable/CS/enumerable.cs	✅Succeeded	View
samples/snippets/visualbasic/VS_Snippets_CLR_System/system.Linq.Enumerable/VB/Enumerable.vb	✅Succeeded	View
xml/System.Linq/Enumerable.xml	✅Succeeded	View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

• Try searching the docs.microsoft.com contributor guides
• Post your question in the Docs support channel

[BillWagner]

ping @KathleenDollard

[matthiaslischka]

Any updates on that? Not sure what's up with the Snippets 5000 check - not related to the change.

[BillWagner]

closing and reopening for a fresh build (I Think the out-of-support versions have been addressed)

[BillWagner]

Hi @matthiaslischka

I looked everything over. I'll approve the changes once we have a clean build.

The issue with the snippets build is that it's using netcoreapp2.2, which is out of support. Can you update that to net6.0? These samples would still compile, and then I'll merge it.

[opbld31]

Docs Build status updates of commit 0ce71d4:

✅ Validation status: passed

File	Status	Preview URL	Details
samples/snippets/csharp/VS_Snippets_CLR_System/system.Linq.Enumerable/CS/enumerable.cs	✅Succeeded	View
samples/snippets/visualbasic/VS_Snippets_CLR_System/system.Linq.Enumerable/VB/Enumerable.vb	✅Succeeded	View
xml/System.Linq/Enumerable.xml	✅Succeeded	View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

• Try searching the docs.microsoft.com contributor guides
• Post your question in the Docs support channel

[learn-build-service-prod]

Learn Build status updates of commit 3e70262:

✅ Validation status: passed

File	Status	Preview URL	Details
snippets/csharp/System.Linq/Enumerable/AggregateTSource/enumerable.cs	✅Succeeded	View
snippets/csharp/System.Linq/Enumerable/AggregateTSource/Enumerable.csproj	✅Succeeded
snippets/visualbasic/VS_Snippets_CLR_System/system.Linq.Enumerable/VB/Enumerable.vb	✅Succeeded	View
snippets/visualbasic/VS_Snippets_CLR_System/system.Linq.Enumerable/VB/Enumerable.vbproj	✅Succeeded
xml/System.Linq/Enumerable.xml	✅Succeeded	View

For more details, please refer to the build report.

For any questions, please:

• Try searching the learn.microsoft.com contributor guides
• Post your question in the Learn support channel

[learn-build-service-prod]

Learn Build status updates of commit f134982:

✅ Validation status: passed

File	Status	Preview URL	Details
snippets/csharp/System.Linq/Enumerable/AggregateTSource/enumerable.cs	✅Succeeded	View
snippets/csharp/System.Linq/Enumerable/AggregateTSource/Enumerable.csproj	✅Succeeded
snippets/visualbasic/VS_Snippets_CLR_System/system.Linq.Enumerable/VB/Enumerable.vb	✅Succeeded	View
snippets/visualbasic/VS_Snippets_CLR_System/system.Linq.Enumerable/VB/Enumerable.vbproj	✅Succeeded
xml/System.Linq/Enumerable.xml	✅Succeeded	View

For more details, please refer to the build report.

For any questions, please:

• Try searching the learn.microsoft.com contributor guides
• Post your question in the Learn support channel