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. <br><div class="gmail_quote">On Mon, 11 May 2015 at 16:08, Michael Meeks <<a href="mailto:michael.meeks@collabora.com">michael.meeks@collabora.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Noel,<br>
<br>
        I just finished cleaning up all the VclBuilder 'makeFoo' constructors -<br>
and will push to master in a sec.<br>
<br>
        I was wondering - would it be possible to have a clang plugin that<br>
checks for all instances of new <type> where type is derived from<br>
OutputDevice ? =)<br>
<br>
        Those new calls should really all be either:<br>
<br>
        a) VclPtr< type >::Create<br>
or<br>
        b) VclPtr< type >( new <type>, SAL_NO_ACQUIRE )<br>
<br>
        I hope the impl. of a) is covered by b) =)<br>
<br>
        It'd be great to know that no-one is leaking Windows as they are<br>
created, and of course that no-one is mis-using SAL_NO_ACQUIRE<br>
inadvertently either =)<br>
<br>
        Any chance ? =)<br>
<br>
        Thanks,<br>
<br>
                Michael.<br>
<br>
--<br>
 <a href="mailto:michael.meeks@collabora.com" target="_blank">michael.meeks@collabora.com</a>  <><, Pseudo Engineer, itinerant idiot<br>
<br>
</blockquote></div>