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

leak-tracker: Detect memory leaks by identifying not disposed or not GCed objects. #3951

Open
polina-c opened this issue Apr 1, 2022 · 8 comments
Assignees
Labels
P2 important to work on, but not at the top of the work list. screen: memory Issues with the Memory screen.
Milestone

Comments

@polina-c
Copy link
Contributor

polina-c commented Apr 1, 2022

Motivation: go/flutter-devtools-survey-2022

@polina-c polina-c added the screen: memory Issues with the Memory screen. label Apr 1, 2022
@polina-c polina-c added this to the M39 milestone Apr 1, 2022
@polina-c polina-c self-assigned this Apr 1, 2022
@polina-c
Copy link
Contributor Author

polina-c commented Apr 1, 2022

@polina-c polina-c assigned terrylucas and unassigned terrylucas Apr 6, 2022
@flutter flutter deleted a comment from terrylucas Apr 6, 2022
@polina-c polina-c changed the title Memory leaks: detect disposed, not GC-ed links Detect memory leaks by identifying not disposed or not GCed objects. Apr 7, 2022
@kenzieschmoll kenzieschmoll modified the milestones: M39, M41 May 5, 2022
@polina-c polina-c added the P3 label May 20, 2022
@polina-c
Copy link
Contributor Author

Options for Flutter specific leak tracker:

  1. package:leak_tracker_flutter published in tools.dart.dev, and banking on 'flutter' in the name being clarifying enough
  2. keeping leak_tracker_flutter in the same mono-repo, but publishing it out of the flutter.dev publisher
  3. putting the code in the flutter/packages mono-repo and using the flutter.dev publisher

@devoncarew leans towards the 2nd and 3rd options

@stuartmorgan
Copy link

2. keeping leak_tracker_flutter in the same mono-repo, but publishing it out of the flutter.dev publisher

It appears we did this; is there discussion/context somewhere on why that option was chosen? AFAIK this is the only flutter.dev-published package that is not in flutter/packages, which makes this pretty confusing.

@polina-c
Copy link
Contributor Author

  1. keeping leak_tracker_flutter in the same mono-repo, but publishing it out of the flutter.dev publisher

It appears we did this; is there discussion/context somewhere on why that option was chosen? AFAIK this is the only flutter.dev-published package that is not in flutter/packages, which makes this pretty confusing.

Here is the issue with discussion: dart-lang/leak_tracker#52 (comment)

@Solido
Copy link

Solido commented Nov 28, 2024

Documentation is explicit about pure Dart packages not being supported.
Will this land in the roadmap or is there a technical roadblock? Thanks!

@polina-c polina-c added P2 important to work on, but not at the top of the work list. and removed P1 high priority issues at the top of the work list, actively being worked on. labels Nov 29, 2024
@polina-c
Copy link
Contributor Author

There is no technical roadblock, but the task to enable pure dart does not have resources at the moment.
Technically, it is easy to support, because the mechanism is the same, it just should be tested and documented. If you are willing to contribute, please, create separate issue and we can discus details.

@polina-c
Copy link
Contributor Author

polina-c commented Dec 6, 2024

Issue: dart-lang/leak_tracker#257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 important to work on, but not at the top of the work list. screen: memory Issues with the Memory screen.
Projects
None yet
Development

No branches or pull requests

5 participants