<div dir="ltr">On 22 December 2012 16:34, Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Nick,<br>
<div class="im"><br>
On Fri, Dec 21, 2012 at 11:01 PM, Nick Lewycky <<a href="mailto:nicholas@mxc.ca">nicholas@mxc.ca</a>> wrote:<br>
> The attached patch adds a flag to control warn_redecl_library_builtin,<br>
> -Wlibrary-redeclaration. I also update Misc/warning-flags.c. Please review!<br>
<br>
</div>Seems like the point of this review is to bikeshed on the warninig<br>
group name, so, here goes.<br></blockquote><div><br></div><div>More or less. I was hoping a clang code owner would express an opinion, and that's what I would commit.<br></div><div><br></div><div style>Nick</div><div style>

 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
-Wlibrary-redeclaration seems to suggest that we will warn on any<br>
redecl of a library function, like "it'd be better to use #include<br>
<foo.h> instead of declaring foo() directly".  I'd suggest<br>
-Wincompatible-library-redeclaration or -Wbad-library-redeclaration.<br></blockquote><div><br></div></div></div></div>