<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 11, 2015 at 10:11 PM, Daniel Marjamäki <span dir="ltr"><<a href="mailto:daniel.marjamaki@evidente.se" target="_blank">daniel.marjamaki@evidente.se</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">danielmarjamaki added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D12359#287522" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12359#287522</a>, @rsmith wrote:<br>
<br>
> Why does this construct justify the compiler emitting a warning? It seems to be reporting a fact about the code rather than a bug, and as there are many coding styles where variables are not routinely marked as const whenever possible, this appears to be checking that the code conforms to a particular coding style. As such, this seems like a better fit as a clang-tidy check than as a compiler warning.<br></span></blockquote><div><br></div><div>I don't think you've responded to this. Why do you think this situation justifies the compiler emitting a warning? This seems outside the normal scope of things that we'd warn on, since we have no proof or strong evidence that the code is buggy or does something other than what the author intended. There are lots of legitimate scenarios when a function might be passed a non-const pointer where it has no intention of modifying the pointee (perhaps the function is a callback or similar and must have a certain signature), and there seems to be no obvious syntactic way to suppress this warning on a case-by-case basis.</div><div><br></div><div>Conversely, this seems very well suited to exist as a clang-tidy check.</div></div></div></div>