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.

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

DisplayInformation on Android leaks memory #2230

Closed
1 of 11 tasks
MartinZikmund opened this issue Dec 1, 2019 · 1 comment
Closed
1 of 11 tasks

DisplayInformation on Android leaks memory #2230

MartinZikmund opened this issue Dec 1, 2019 · 1 comment
Labels
kind/bug Something isn't working platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform

Comments

@MartinZikmund
Copy link
Member

MartinZikmund commented Dec 1, 2019 •
edited
Loading

Current behavior

WindowManager and DisplayMetrics instances are not disposed of after use, which causes memory leaks on Android.

Expected behavior

No memory leaks.

How to reproduce it (as minimally and precisely as possible)

Properties are re-initialized when display configuration changes (for example on orientation change). When this is done repeatedly, memory footprint is growing.

Environment

Nuget Package: latest

Package Version(s): latest

Affected platform(s):

  • iOS
  • Android
  • WebAssembly
  • WebAssembly renderers for Xamarin.Forms
  • Windows
  • Build tasks
  • Solution Templates

Visual Studio:

  • 2017 (version: )
  • 2019 (version: )
  • for Mac (version: )

Relevant plugins:

  • Resharper (version: )

Anything else we need to know?
@MartinZikmund MartinZikmund added kind/bug Something isn't working triage/untriaged Indicates an issue requires triaging or verification labels Dec 1, 2019
@MartinZikmund MartinZikmund mentioned this issue Dec 1, 2019
Merged
7 tasks
@MartinZikmund MartinZikmund added platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform and removed triage/untriaged Indicates an issue requires triaging or verification labels Dec 1, 2019
@MartinZikmund
Copy link
Member Author

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Something isn't working platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@MartinZikmund