<div dir="ltr">My vote is for this:<div><br></div><div>> <span style="font-family:arial,sans-serif;font-size:13px">I think there should be a WERROR mode in the CMake build, and build bots should run with it. That way releases etc aren't impacted by new warnings, but developers *do* notice when they introduce a warning.</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><font face="arial, sans-serif">In particular, warning with clang.</font></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br>
</span></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div>With the caveat of this:</div><div><br></div><div>> <span style="font-family:arial,sans-serif;font-size:13px">You can easily disable warnings that don't have value. We do this all the time in LLVM.</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">where there are likely a handful of warnings I'd definitely vote for turning off (like the full-coverage-enum case with default present). We can discuss these as we break warnings that we think are worthwhile (and fix them) or not (and then disable the warning).</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">-Todd</span></div><div class="im" style="font-family:arial,sans-serif;font-size:13px">
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 15, 2014 at 7:05 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</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">In the clang/llvm world, I often get a small handful of warnings when building locally, almost always due to my local compiler having buggy warnings. If -Werror was enabled by default in the clang/llvm build, I'd have a broken build fairly often.<div>
<br></div><div>Maybe we could detect compiler versions and only enable -Werror for compilers that are in some whitelist.</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Aug 14, 2014 at 5:42 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.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">There are alot of compilers, but only a small number that we support building with. And really, the burden would only be on making sure that the compiler you personally build with builds warning free (which, unfortunately, even that doesn't always happen). If someone checks something in on Mac that introduces warnings on Windows with MSVC, then a Windows person would need to fix it, but I don't think that's too big of a deal (well, I don't mind anyway). The difference is just that "strive" would become "require". Sometimes you can't fix warnings for whatever reason, but you could still disable them in that case.<div>
<br></div><div>I guess in an ideal world I would agree with you, that we should strive to keep warnings down to as low a level as possible. But in practice I think it's too easy to ignore them, and so people do.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote"><div>On Thu, Aug 14, 2014 at 5:28 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>
</div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There are so many compilers and each compiler is good at detecting things that other compilers don't warn about.<br>
<br>
So I don't think we should enable warnings as errors, but I do think we should strive to keep warnings down to as low a level as possible.<br>
<span><font color="#888888"><br>
Greg<br>
</font></span><div><div><br>
> On Aug 14, 2014, at 10:38 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
><br>
> What would it take to enable warnings as errors on all platforms?<br>
><br>
> I've done alot of work getting the windows build warning free, but it seems like very time I sync the code on linux or Mac, there's many many new warnings introduced. I'm always vigilant about fixing warnings in my own code, it would be nice if everyone else had the same level of vigilance. If warnings broke the build, then I think that would be a good motivator to fix your warnings.<br>
><br>
> Thoughts?<br>
</div></div><div><div>> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br>
</div></div></blockquote></div></div><br></div>
<br>_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>