<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 28, 2014 at 1:26 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I would have thought that the difference is that a remark indicates<br>

something that *isn't* a problem, and is just informational. That being the<br>
case, it doesn't make sense to me to upgrade remarks to warnings or to<br>
errors. It would seem bizarre to me if -Werror converts remarks to errors<br>
(and indeed, in this patch, it does not).<br>
</blockquote>
<br></div>
Good point. I explicitly mentioned that '-Werror' does not trigger on remarks in the commit message.<div class=""><br>
<br>
> I think we should also reject<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-Werror=some-remark.<br>
</blockquote>
<br></div>
Several people mentioned that they like this feature including Chris. This either needs more discussion or just more experience when some<br>
remarks have been added.</blockquote></div><br>I actually really agree with Richard here. If these were things that made sense to "escalate" they would be warnings. I think a good metric for whether something should be a warning or a remark is whether it really ever makes sense to break the compilation on it because it indicates a problem.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I also think that it is ultimately a mistake to re-use the same commandline syntax for remarks. While I'd rather re-use all the infrastructure, I'd rather separate these into a different spelling eventually.</div>
</div>