Extract array of const to static readonly field #33794

Activity

1. terrajobst

   added

   api-suggestionEarly API idea and discussion, it is NOT ready for implementationEarly API idea and discussion, it is NOT ready for implementation

   area-System.Runtime

   code-analyzerMarks an issue that suggests a Roslyn analyzerMarks an issue that suggests a Roslyn analyzer

   on Mar 19, 2020
2. Dotnet-GitSync-Bot

   added

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   on Mar 19, 2020
3. jeffhandley

   added this to the 5.0 milestone on Mar 20, 2020
4. jeffhandley

   added

   code-fixerMarks an issue that suggests a Roslyn code fixerMarks an issue that suggests a Roslyn code fixer

   on Mar 21, 2020
5. jeffhandley commented on Mar 21, 2020

   

   jeffhandley

   on Mar 21, 2020

   Member

   More actions

   Estimates:

   • Analyzer: Small
   • Fixer: Large
6. terrajobst

   added

   api-approvedAPI was approved in API review, it can be implementedAPI was approved in API review, it can be implemented

   and removed

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   api-suggestionEarly API idea and discussion, it is NOT ready for implementationEarly API idea and discussion, it is NOT ready for implementation

   on Mar 21, 2020
7. jeffhandley commented on May 26, 2020

   

   jeffhandley

   on May 26, 2020

   Member

   More actions

   Moving this one out of .NET 5, but this would be a good warmup analyzer for someone, and it would be a high-value analyzer in .NET 6 because it applies to some recently added APIs.
8. jeffhandley

   added

   good first issueIssue should be easy to implement, good for first-time contributorsIssue should be easy to implement, good for first-time contributors

   on May 26, 2020
9. jeffhandley

   modified the milestones: 5.0, Future on May 26, 2020
10. jeffhandley commented on May 26, 2020

    

    jeffhandley

    on May 26, 2020

    Member

    More actions

    The hardest part of this one will be coming up with the list of APIs we want this to target--using a heuristic is the recommended approach. Beyond that, we think the analyzer will be pretty easy; the fixer will have some edge cases to consider for how to rewrite the code to reflect the definition/usage of the const--simply extracting the const won't necessarily be straightforward, and there will need to be an ordered list of approaches to apply.
11. 5 remaining items

    Load more

12. teo-tsirpanis commented on Dec 28, 2020

    

    teo-tsirpanis

    on Dec 28, 2020

    Contributor

    More actions

    Such optimization could be automatically done by the compiler for methods that accept a ReadOnlySpan.

    👍React with 👍1Reacted by Simon Felix
13. tannergooding

    modified the milestones: 6.0.0, Future on Jul 12, 2021
14. steveberdy commented on Jul 20, 2021

    

    steveberdy

    on Jul 20, 2021

    Contributor

    More actions

    @buyaa-n I would like this assigned to me. Thanks.

    👍React with 👍1Reacted by Buyaa Namnan
15. buyaa-n

    assigned

    steveberdy

    on Jul 20, 2021
16. jeffhandley

    modified the milestones: Future, 7.0.0 on Aug 20, 2021
17. buyaa-n

    removed

    help wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributors

    on Sep 1, 2021
18. carlossanlop

    added

    in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

    on Nov 23, 2021
19. jeffhandley

    modified the milestones: 7.0.0, Future on Jul 9, 2022
20. buyaa-n

    modified the milestones: Future, 8.0.0 on Nov 9, 2022
21. buyaa-n

    closed this as completedin roslyn-analyzers#5383on May 8, 2023
22. ghost locked as resolved and limited conversation to collaborators on Jun 7, 2023