<div dir="ltr">(it's a bit easier if you include the test in the same patch file - also 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 pointer, should you have a test case that exercises that on a > 1 level of depth? Demonstrate that we warn on both levels (if that's the right thing to do?)?<br><br>Optionally (probably in a separate follow-up patch) you could add a note with a fixit to include the missing consts.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 11, 2014 at 10:58 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">Hi,<br>
<br>
I implemented -Wcast-qual. The patch is actually quite short (attached + a test<br>
case).<br>
<br>
This fixes #13772 and also note that -Wcast-qual is used in llvm build itself.<br>
<br>
Is this ok to be commited? Thanks<br>
<span class="HOEnZb"><font color="#888888"><br>
Roman<br>
<br>
</font></span><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></blockquote></div><br></div>