Skip to content
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

DoNotEnumerateCollectionInLinqChainAnalyzer: investigate different collection types #33

Open
ronimizy opened this issue May 1, 2024 · 0 comments

Comments

@ronimizy
Copy link
Member

ronimizy commented May 1, 2024

Analyzer is probably working incorrect for types that are not just memory mapping of IEnumerable (ie Dictionary, Lookup, HashSet). As these types are doing some logic on enumerable values, just removing them can be incorrect.

We should investigate on correct and incorrect cases, and provide proper code fixes, that won't change logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant