Wpf: Why we custom the HashSet

[lindexi]

As we can see, we define a custom HashSet . Is this necessary?

https://github.com/dotnet/wpf/blob/69545415380cce83e195dd1a8883a9d312f76e16/src/Microsoft.DotNet.Wpf/src/System.Xaml/System/Xaml/Context/HashSet.cs#L10

[weltkante]

HashSet didn't exist in .NET Framework 3.0 when WPF initially was released, so this probably can be replaced by the framework HashSet now.

[lindexi]

The Dictionary exist in .NET Framework 2.0

[lindexi]

@weltkante Do you want to do this?

[weltkante]

Do you want to do this?

Only if I'm asked to, doesn't seem a complicated PR or urgent to fix, also the team seems still busy with reorganizing itself. I'm more active in the WinForms repo at the moment.

[miloush]

Note that the framework's HashSet<T> derives neither from Dictionary nor IDictionary, so the code consuming it will need to be changed as well.

There is also similar but different System.Xaml.XamlObjectReader.HashSet<T>.