Skip to content

Fix TextPainter leaking#149

Closed
tmpsantos wants to merge 1 commit intosimc:masterfrom
tmpsantos:tmpsantos-fix_memory_leak
Closed

Fix TextPainter leaking#149
tmpsantos wants to merge 1 commit intosimc:masterfrom
tmpsantos:tmpsantos-fix_memory_leak

Conversation

@tmpsantos
Copy link
Copy Markdown

@tmpsantos tmpsantos commented Oct 2, 2024

Dispose it when it is no longer in use.

Also add leak detection to all the unit tests to prevent regressions.

Fixes #150

Dispose it when it is no longer in use.

Also add leak detection to all the unit tests to prevent regressions.
@tmpsantos
Copy link
Copy Markdown
Author

Removing both dispose() will make the tests fails with the leak error.

@tmpsantos
Copy link
Copy Markdown
Author

@simc do you have any feedback on this PR?

@Nana-Kwame-bot
Copy link
Copy Markdown

Is this going to be merged ever?

@tmpsantos
Copy link
Copy Markdown
Author

@simc could you please take a look? Anything I could help you here with?

@FaFre
Copy link
Copy Markdown

FaFre commented Feb 23, 2025

Since this repo has been unmaintained for a couple of years now, I decided to fork it and merge all the outstanding PR's: https://github.com/FaFre/auto_size_text/blob/master/CHANGELOG.md

The package also has been pushed to pub: https://pub.dev/packages/flutter_auto_size_text

@tmpsantos tmpsantos closed this Mar 28, 2026
@tmpsantos tmpsantos deleted the tmpsantos-fix_memory_leak branch March 28, 2026 21:49
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

Successfully merging this pull request may close these issues.

Memory leak on TextPainter

3 participants