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