Menu -> VclPtr ...

[Michael Meeks]

Hi Noel,

	Had a quick read of:

	https://gerrit.libreoffice.org/#/c/26516/

	Looks rather good to me - all the instances which you highlight as
'dodgy' look really odd to me even in the pre-existing code ;-) deleting
items that still have private pointers lurking around in child
containers is ... ;-) 'interesting'.

	"delete pFoo->LetMePokeInYourInnards()"

	is a curious use of pFoo ;-)

	What do you think of the attached to fix that ? (not compile tested
sadly - but hopefully squashes nicely and achieves what we want ?

	Would love to have some of that automated UI testing stuff implemented
at this stage - but absent that, if it works with some manual testing -
it'd be good to get in I think; while an impressive change - it's far
smaller and less disruptive than the original VclPtr stuff I think =)

	ATB,

		Michael.

-- 
michael.meeks at collabora.com <><, GM Collabora Productivity
 Skype: mmeeks, Google Hangout: mejmeeks at gmail.com
 (M) +44 7795 666 147 - timezone usually UK / Europe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: de-dogify.diff
Type: text/x-patch
Size: 4445 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/libreoffice/attachments/20160622/5550111a/attachment.bin>