<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 8, 2016 at 8:37 AM Benjamin Kramer <<a href="mailto:benny.kra@gmail.com">benny.kra@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-Wmissing-prototype only warns for functions, I want to catch classes<br>
too.</blockquote><div><br></div><div>Ah, fair enough. Yeah, a clang-tidy check for things in the global namespace that are in a main file rather than a header could be another pivot there.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Also functions in the global namespace with a prototype are still<br>
badness in some coding styles. </blockquote><div><br></div><div>*nod* makes sense - especially in C++, where, as you say, you might want to bless the extern "C" function declarations only.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The limitation on definitions was to<br>
cut down on false positives, the current version of the patch doesn't<br>
have that limitation but I'm pondering on putting it back as there are<br>
too many false positives.<br>
<br>
On Mon, Aug 8, 2016 at 5:34 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br>
> This seems to have a lot of overlap with -Wmissing-prototype, really - what<br>
> do you think of the overlap/distinction between the two?<br>
><br>
> On Wed, Aug 3, 2016 at 1:25 PM Eugene Zelenko via cfe-commits<br>
> <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> Eugene.Zelenko added a subscriber: Eugene.Zelenko.<br>
>> Eugene.Zelenko added a comment.<br>
>><br>
>> Please mention this check in docs/ReleaseNotes.rst (in alphabetical<br>
>> order).<br>
>><br>
>><br>
>> <a href="https://reviews.llvm.org/D23130" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23130</a><br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>