<div dir="ltr"><div>Hi,<br><br>I have finally implemented the same checker as a clang-tidy check if anyone is interested. The patch is attached.<br><br></div>Cheers,<br><div><div><div><div>--<br>Beren Minor</div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 22, 2015 at 7:01 PM, Beren Minor <span dir="ltr"><<a href="mailto:beren.minor@gmail.com" target="_blank">beren.minor@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>No particular reason except the lack of knowledge. I didn't know there was a tool that could be easily extended.<br>Initially it was just a sample code to play with the LibTooling API without any prior knowledge, and it's still very simple.</div><br></div>If the idea of such checks is popular, then I could happily offer to implement it as clang-tidy checks.<br>--<br>Beren Minor</div><div><div><div><div class="gmail_extra">
<br><div class="gmail_quote">On Mon, Jun 22, 2015 at 6:43 PM, Meador Inge <span dir="ltr"><<a href="mailto:meadori@gmail.com" target="_blank">meadori@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On Mon, Jun 22, 2015 at 11:14 AM, Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>> wrote:<br>
> Any reason this is not done as clang-tidy check (which also supports<br>
> fixits)?<br>
<br>
</span>FWIW, I had a similar thought.  At first glance, that does seem to be<br>
a better place for it.<br>
<span><font color="#888888"><br>
-- Meador<br>
</font></span></blockquote></div><br></div></div></div></div></div>
</blockquote></div><br></div></div>