> I really hate the ref/deref happening on QML side, could we use QML ownership
> to do this?
I think I've addressed all your concerns. Was able to move the model instantiation into the cache now that i've fixed up the QMenuModel mocking.
The GC looks a bit hairy in the qmltests for testing the deletion, but it seems to work.
> I really hate the ref/deref happening on QML side, could we use QML ownership
> to do this?
I think I've addressed all your concerns. Was able to move the model instantiation into the cache now that i've fixed up the QMenuModel mocking.
The GC looks a bit hairy in the qmltests for testing the deletion, but it seems to work.