search

xamarin / Xamarin.Forms

Xamarin.Forms is no longer supported. Migrate your apps to .NET MAUI.
https://aka.ms/xamarin-upgrade
Other
5.64k stars 1.88k forks source link

[Bug] XF.UWP CollectionView memoryleak #15717

Open sjorsmiltenburg opened 1 year ago

sjorsmiltenburg commented 1 year ago

Description

Hi, is this fix for MAUI also going to be added to Xamarin.Forms?

I recently reported a bug that is so similar that it is probably the same cause / fix.

Steps to Reproduce

  1. See Bugreport for reproducable sample project on XF.UWP

Expected Behavior

no memoryleak

Actual Behavior

memoryleak

Basic Information

Environment

Show/Hide Visual Studio info ``` ```

Build Logs

Screenshots

Reproduction Link

Workaround

jfversluis commented 1 year ago

Will see if I can do that. If it's important to you, happy to have your PR as well!

MitchBomcanhao commented 1 year ago

just a sidenote - In your repro project you're using a xamarin forms package (5.0.0.2196) from october 2021. is it still as bad on the current version?

sjorsmiltenburg commented 1 year ago

@MitchBomcanhao yes, with an update to the latest version 5.0.0.2578 the problem still persists. With a few resizes the mem usage of the sample app grows from 100 mb to 300+ and beyond. I have updated the repo.

sjorsmiltenburg commented 1 year ago

Will see if I can do that. If it's important to you, happy to have your PR as well!

The problem is quite pressing for me. I would love to submit a PR, I tried to invest some time into it and getting the framework to build/run on my machine takes me too much time. (I'm soo busy with other stuff and i'm quite out of my comfort zone here). So for me it almost makes more sense to invest my time in migrating to Maui as to create a PR to fix this problem for myself.