[Bug 63154] replace tools/solar.h macros with osl versions

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=63154

--- Comment #32 from Michael Meeks <michael.meeks at collabora.com> ---
There are 600 or so hits of DELETEZ, in each case we'd need to work out whether
the memory could be re-used inadvertently and I guess add a manual NULL as/when
we think it could be; that my be somewhat hard to verify of course.

I'm not against that, clearly any usage of DELETEZ of a member in a destructor
is bogus eg.

accessibility/source/extended/accessibletabbarbase.cxx-

AccessibleTabBarBase::~AccessibleTabBarBase()
{
    ClearTabBarPointer();
    DELETEZ( m_pExternalLock );
}

or

cui/source/tabpages/macroass.cxx

_SfxMacroTabPage::~_SfxMacroTabPage()
{
    DELETEZ( mpImpl );
}

And those can be removed safely. I guess we should consult some C++ guru - who
(no doubt) would recommend one of the bewildering array of templatized smart
pointers of varying efficiency that can magically do something similar :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20131102/34602c21/attachment.html>