<div dir="ltr">[+Richard for oversight]<br><br>char **y1 = (char **)ptrptr;<span class="" style="white-space:pre">        </span>// expected-warning {{cast from 'const char *const *' to 'char **' drops const qualifier}} expected-warning {{cast from 'const char *const' to 'char *' drops const qualifier}}<br><br>I think if we're going to warn on multiple layers (I'm not sure that's ideal - is that consistent with GCC's warning? Does GCC warn on mismatched types too - "const T*" -> "U*"? - do we warn there too, or only when there's a valid implicit conversion like the void* example?) then we should probably drop the top level const, "const char *const" -> "char*" - the top level const on the first type is confusing/misleading, it's only relevant to show "const char*" and "char*".<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 12, 2014 at 10:41 AM, Roman Divacky <span dir="ltr"><<a href="mailto:rdivacky@vlakno.cz" target="_blank">rdivacky@vlakno.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I expanded the testcase and fixed the grammar in the actual warning.<br>
<br>
New patch attached.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Nov 11, 2014 at 05:03:33PM -0800, David Blaikie wrote:<br>
> (it's a bit easier if you include the test in the same patch file - also<br>
> you can use Phabricator if you like - some reviewers perefer it)<br>
><br>
> Since you've got the loop there for seeing through multiple levels of<br>
> pointer, should you have a test case that exercises that on a > 1 level of<br>
> depth? Demonstrate that we warn on both levels (if that's the right thing<br>
> to do?)?<br>
><br>
> Optionally (probably in a separate follow-up patch) you could add a note<br>
> with a fixit to include the missing consts.<br>
><br>
> On Tue, Nov 11, 2014 at 10:58 AM, Roman Divacky <<a href="mailto:rdivacky@vlakno.cz">rdivacky@vlakno.cz</a>> wrote:<br>
><br>
> > Hi,<br>
> ><br>
> > I implemented -Wcast-qual. The patch is actually quite short (attached + a<br>
> > test<br>
> > case).<br>
> ><br>
> > This fixes #13772 and also note that -Wcast-qual is used in llvm build<br>
> > itself.<br>
> ><br>
> > Is this ok to be commited? Thanks<br>
> ><br>
> > Roman<br>
> ><br>
> ><br>
> > _______________________________________________<br>
> > cfe-commits mailing list<br>
> > <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
> ><br>
> ><br>
</div></div></blockquote></div><br></div>