- Notifications
You must be signed in to change notification settings - Fork749
Multiple fixes related to Dictionary.Keys bug#1786
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
…the value should not be disposed
…eneric arguments as long as there is no ambiguity
Could you split part 2 out into a separate PR for discussion and add a few tests? |
lostmsu commentedMay 9, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Do you have any specific concerns? The reason I made the quality of life change is to use it in |
Uh oh!
There was an error while loading.Please reload this page.
What does this implement/fix? Explain your changes.
This has individual commits for:
Dictionary.Keys
to be confused with theDictionary
itselfGeneric.IEnumerable(obj)
, which previously required the generic type argument to be specified. E.g.Generic.IEnumerable[String](obj)
was requiredDoes this close any currently open issues?
fixes#1785
Checklist
Check all those that are applicable and complete.
CHANGELOG
@Martin-Molinero