Skip to content

[analysis server] Refactor/quick fix to replace all usages of typedef with represented type #52846

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.

Sign up for GitHub

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

Jump to bottom
Open
Tracked by #55825
parlough opened this issue Jul 4, 2023 · 1 comment
Open
Tracked by #55825

[analysis server] Refactor/quick fix to replace all usages of typedef with represented type #52846

parlough opened this issue Jul 4, 2023 · 1 comment
Labels
area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. devexp-refactoring Issues with analysis server refactorings P3 A lower priority bug or feature request type-enhancement A request for a change that isn't a bug

Comments

@parlough
Copy link
Member

parlough commented Jul 4, 2023

I've had a few times where I started with a typedef for ease of writing or thinking it would be useful, but later determined it wasn't worth it or just added extra confusion.

It would be useful if there was a code action to convert all usages of the typedef to the represented type. This could or could not remove the typedef declaration as well.
@vsmenon vsmenon added the legacy-area-analyzer Use area-devexp instead. label Jul 5, 2023
@scheglov scheglov added P3 A lower priority bug or feature request devexp-refactoring Issues with analysis server refactorings type-enhancement A request for a change that isn't a bug labels Jul 5, 2023
@scheglov
Copy link
Contributor

scheglov commented Jul 5, 2023

This seems to be similar to Inline method, so maybe something like Inline typedef.

@mosuem mosuem mentioned this issue May 23, 2024
Open
26 tasks
@bwilkerson bwilkerson added area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. and removed legacy-area-analyzer Use area-devexp instead. labels Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. devexp-refactoring Issues with analysis server refactorings P3 A lower priority bug or feature request type-enhancement A request for a change that isn't a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@scheglov @vsmenon @bwilkerson @parlough