-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Potential addition of an OrderedDictionary<TKey, TValue>
type
#220
Comments
It's MIT licensed so please do with it what you will. Personally, outside of including it in the BCL I think it would probably be used more in a more collection focused library. I think more people would be able to discover its existence and would be more willing to take a dependency on it than a more general purpose library like this. -Edit (After re-reading this I guess I really like the word "more" 😄) |
The only general purpose collections library that immediately comes to mind here is C5, but that seems to be unmaintained these days... 🤔 One could also imagine a DotNext.Collections package, but at that point I might be asking too much of @sakno. 👀 |
On the one hand, I never tried to reach the goal to incorporate reusable data structures to .NEXT such as trees, collections, etc. From the other side, the library has specialized collections such as typed maps, concurrent cache, etc. Also, there is very similar discussion and proposal: #163. And it was rejected. |
If you don't think it belongs in .NEXT, that's totally fine. This was just the first possibility that came to mind while trying to think of a potential home for it. |
Let's keep this issue open for now. I don't want to be a sole decision maker, .NEXT is an open-source project and community opinion matters. |
Still has to go through API review, but I think it's safe to say that this won't be needed after all: dotnet/runtime#24826 (comment) |
Finally |
Context: dotnet/runtime#24826 (especially the last 10 or so comments)
Do you think it would make sense for .NEXT to include @TylerBrinkley's
OrderedDictionary<TKey, TValue>
implementation inDotNext.Collections
? (Assuming Tyler takes no issue with this, of course.) It seems like there's high demand for this type but low appetite for prioritizing it from the .NET team, and it's only currently available in an unmaintained, pre-release, and deprecated package.The text was updated successfully, but these errors were encountered: