I’ve just reviewed the PR and I really didn’t see any memory leaks. At most, what happens is that a new widget is created every time Display is called instead of reusing the one that’s already instantiated.
In practice, this is not really an issue, because the object is a shared_ptr coming from...