vcl/README.lifecycle | 31 ++++++++++++++++++++++++++++--- 1 file changed, 28 insertions(+), 3 deletions(-)
New commits: commit 2932d2db599c09ecce3faa2d627e9ee4f251183a Author: Michael Meeks <[email protected]> Date: Tue Apr 28 22:02:47 2015 +0100 More VclPtr documentation. Change-Id: I8b375ba0eab97c4e0383ed6638b9dc4be149369e diff --git a/vcl/README.lifecycle b/vcl/README.lifecycle index c4ca67a..daa5fbb 100644 --- a/vcl/README.lifecycle +++ b/vcl/README.lifecycle @@ -156,11 +156,36 @@ or: ---------- Who owns & disposes what ? ---------- -** referencing / ownership inheritance / hierarchy. + Window sub-classes tend to create their widgets in one of two +ways and often both. -** VclBuilder - + and it's magic dispose method. + 1. Derive from VclBuilderContainer. The VclBuilder then owns + many of the sub-windows, which are fetched by a 'get' + method into local variables often in constructors eg. + VclPtr<PushButton> mpButton; // in the class + , get(mpButton, "buttonName") // in the constructor + mpButton.clear(); // in dispose. + + We only clear, not disposeAndClear() in our dispose method + for this case, since the VclBuilder / Container truly owns + this Window, and needs to dispose its hierarchy in the + right order - first children then parents. + + 2. Explicitly allocated Windows. These are often created and + managed by custom widgets: + + VclPtr<ComplexWidget> mpComplex; // in the class + , mpComplex( VclPtr<ComplexWidget>::Create( this ) ) // constructor + mpComplex.disposeAndClear(); // in dispose + + ie. an owner has to dispose things they explicitly allocate. + + In order to ensure that the VclBuilderConstructor + sub-classes have their Windows disposed at the correct time + there is a disposeBuilder(); method - that should be added + -only- to the class immediately deriving from + VclBuilderContainer's dispose. ---------- What remains to be done ? ---------- _______________________________________________ Libreoffice-commits mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
