VclPtr / Builder cleanup ...
Noel Grandin
noelgrandin at gmail.com
Mon May 11 11:11:20 PDT 2015
Sure no problem. My RenderContext plugin isn't being as useful as hoped
anyway. I don't seem to have a good enough handle on that yet to build a
decent set of rules.
On Mon, 11 May 2015 at 16:08, Michael Meeks <michael.meeks at collabora.com>
wrote:
> Hi Noel,
>
> I just finished cleaning up all the VclBuilder 'makeFoo'
> constructors -
> and will push to master in a sec.
>
> I was wondering - would it be possible to have a clang plugin that
> checks for all instances of new <type> where type is derived from
> OutputDevice ? =)
>
> Those new calls should really all be either:
>
> a) VclPtr< type >::Create
> or
> b) VclPtr< type >( new <type>, SAL_NO_ACQUIRE )
>
> I hope the impl. of a) is covered by b) =)
>
> It'd be great to know that no-one is leaking Windows as they are
> created, and of course that no-one is mis-using SAL_NO_ACQUIRE
> inadvertently either =)
>
> Any chance ? =)
>
> Thanks,
>
> Michael.
>
> --
> michael.meeks at collabora.com <><, Pseudo Engineer, itinerant idiot
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20150511/0f3d0927/attachment.html>
More information about the LibreOffice
mailing list