<div dir="ltr">Awesome! Thanks.<div><br></div><div>I'll go ahead and factor out a checker and submit it for review.</div><div><br></div><div>/Eric</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 10, 2018 at 7:28 AM, Jonas Toth via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> I think there's already precedent for adding this sort of thing to<br>
> clang-tidy. For instance, we have the LLVM module which is largely<br>
> only useful to people working on the LLVM code base. I see no problem<br>
> with adding a module for libc++ developers as well. My preference<br>
> would be to name the new module "libcxx-module" and prefix the checks<br>
> with "libcxx-".<br>
><br>
> ~Aaron<br>
><br>
</span>Yes. There is even google and zircon/fuchsia specific checks, that are<br>
company/project specific.<br>
<br>
And a potential mechanism to dynamically load clang-tidy plugins is<br>
probably loading the current modules.<br>
I think adding a module now and doing a potential port to a plugin<br>
infrastructure is ok not too much effort.<br>
<br>
Best, Jonas<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div>