Dinamicaly adding/removing MenuItem-s in Menu leads to memory leaks about avalonia HOT 8 OPEN

After a little researche, I found that subscription is cached in _innerSerialDisposable field of class Switch<T>._. When MenuItem is added or removed, VisualParentProperty of it changes. Each change of property calls OnNext method of Switch<T>._. Each call creates new instance of InnerObserver and stores it in _innerSerialDisposable, but do nothing with old value. I just simply added _innerSerialDisposable.Dispose call befo setting new value, and now memory leak is gone.

        public override void OnNext(IObservable<TSource> value)
        {
            ulong id;

            lock (_gate)
            {
                id = unchecked(++_latest);
                _hasLatest = true;
            }

            var innerObserver = new InnerObserver(this, id);

            _innerSerialDisposable?.Dispose();  // <== Added Dispose call before setting new value.
            _innerSerialDisposable = innerObserver;
            innerObserver.Disposable = value.Subscribe(innerObserver);
        }

from avalonia.

@ds1709 probably worth to file a PR

from avalonia.

@timunie I need this fix in version 11.0.*. So, do I need to create two PR, one in branch 11.0 and another in develop?

from avalonia.

@ds1709 for 11.0 this requires probably a support agreement as we are only backporting for upcomming 11.1 at this point of time.

from avalonia.

@timunie is there any problem with support 11.0 version? This fix is not complex, so release 11.0.11 should be easy.

from avalonia.

Limited resources. Nothing else.

from avalonia.

Is there any chance to get this fix in version 11.0? And another question, when is it planned to release version 11.1 (no beta)? According to the commot graph, there are a big amount of improvements from release 11.1.0-beta2. Same for 11.0.10, there are improvements, that wasn't released.

from avalonia.

1. You can request 11.0 backports if you have paid support. Reach out via website if you are interested in general
2. 11.0.10 is actually relesed. You may need to pull templates again
3. 11.1 should drop soonish if there's no regression found while rc testings. No actual due date

Thanks for your understanding

from avalonia.